Revisit CVE-2022-23121, CVE-2022-23123 regression fixes by @andychen-syno and @anodos325

[dgsga]

• Added guard check before access ad_entry()
• Allow zero length entry in compliance with AppleDouble specification
• Avoid setting adouble entries on symlinks

[ghost]

@rdmark, this commit implements @andychen-syno's original 2022 CVE patches along with @anodos325's patch to avoid setting adouble entries on symlinks. This gives the best of both worlds with a much cleaner log, especially on classic Mac clients.

[sonarqubecloud[bot]]

Quality Gate passed

Issues
4 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarCloud

[rdmark]

@dgsga Sorry it's taking a while to wrap my head around this changeset. I trust your judgement of course, but I also want to internalize the logic changes.

[ghost]

No worries, if you find any flaws please feed back. This PR is definitely optional!

[ghost]

Reverted this PR to draft as want to do more research...