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)
Wed, Jan 22, 11:59 AM
Unknown Object (File)
Jan 15 2025, 8:19 PM
Unknown Object (File)
Dec 1 2024, 6:25 AM
Unknown Object (File)
Nov 15 2024, 9:34 AM
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
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 :)