RTEMS | libcsupport: Take two upstream realpath() fixes from FreeBSD (!1444)

[Sam Price (@TheSamPrice)]

Sam Price commented on a discussion: https://gitlab.rtems.org/rtems/rtos/rtems/-/merge_requests/1444#note_157871


I was running `gcc -fanalyze` and wanted these bugs to be fixed, being from an upstream source I went there first looking for fixes.

This is not a 1-1 PR from upstream.
RTEMS had the extra pattern of 
```
if (m)
    free(resolved);
```

Not being a submodule I don't know how to cherry-pick the upstream commits.

I agree the whole upstream commits should be audited. 
Perhaps you store the hash of the original port in a comment at the top of the file, or a change log?
That would allow for periodic audits of upstream sources?  Possibly scripted in some way?

-- 
View it on GitLab: https://gitlab.rtems.org/rtems/rtos/rtems/-/merge_requests/1444#note_157871
You're receiving this email because of your account on gitlab.rtems.org. Unsubscribe from this thread: https://gitlab.rtems.org/-/sent_notifications/5-az7679dxb7q07np9rb12vulro-1d/unsubscribe | Manage all notifications: https://gitlab.rtems.org/-/profile/notifications | Help: https://gitlab.rtems.org/help


-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.rtems.org/pipermail/bugs/attachments/20260825/e778e09f/attachment.htm>