Page MenuHomeFreeBSD

pkg: Allocate a suitably-sized string for the local ABI
ClosedPublic

Authored by jhb on Nov 15 2023, 6:37 PM.
Tags
None
Referenced Files
Unknown Object (File)
Fri, Oct 2, 7:07 PM
Unknown Object (File)
Fri, Oct 2, 6:30 PM
Unknown Object (File)
Thu, Oct 1, 1:01 PM
Unknown Object (File)
Wed, Sep 30, 7:39 AM
Unknown Object (File)
Wed, Sep 16, 9:38 PM
Unknown Object (File)
Wed, Sep 9, 3:25 AM
Unknown Object (File)
Tue, Sep 8, 8:37 AM
Unknown Object (File)
Mon, Sep 7, 8:26 PM
Subscribers

Details

Summary

Previously the local ABI string was written to an on-stack buffer and
the pointer to that buffer was saved in a global before the function
returned. This had two issues: c[ABI].val pointed to a
no-longer-valid on-stack buffer after config_init returned, and the
string could potentially be truncated. Fix both of those by changing
pkg_get_myabi to return a pointer to a string allocated by asprintf.

Reported by: GCC 13 -Wdangling-pointer

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Not Applicable
Unit
Tests Not Applicable