Page MenuHomeFreeBSD

pkg: Use a global buffer for local ABI string
AbandonedPublic

Authored by jhb on Nov 13 2023, 10:30 PM.
Tags
None
Referenced Files
Unknown Object (File)
Sep 24 2024, 5:00 PM
Unknown Object (File)
Sep 23 2024, 4:15 PM
Unknown Object (File)
Sep 23 2024, 9:30 AM
Unknown Object (File)
Sep 22 2024, 8:12 PM
Unknown Object (File)
Sep 22 2024, 6:44 AM
Unknown Object (File)
Sep 21 2024, 11:55 AM
Unknown Object (File)
Sep 21 2024, 11:53 AM
Unknown Object (File)
Sep 16 2024, 6:15 AM
Subscribers

Details

Summary

Otherwise c[ABI].val can point to a no-longer-valid on-stack buffer
after config_init returns.

Reported by: GCC 13 -Wdangling-pointer

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Skipped
Unit
Tests Skipped
Build Status
Buildable 54407
Build 51297: arc lint + arc unit

Event Timeline

jhb requested review of this revision.Nov 13 2023, 10:30 PM
This revision is now accepted and ready to land.Nov 14 2023, 12:24 AM

OK with me, or we could have pkg_get_myabi asprintf it?

OK with me, or we could have pkg_get_myabi asprintf it?

I don't know that we really need that extra complexity TBH. But alternatively, yes, I would make pkg_get_myabi() return an allocated string that the caller has to manage the lifetime of.

Actually, I like the asprintf version better, so I will upload that in a bit.

Sorry for making more work for you :)