HomeFreeBSD

libc/locale: Fix races between localeconv(3) and setlocale(3)

Description

libc/locale: Fix races between localeconv(3) and setlocale(3)

Each locale embeds a lazily initialized lconv which is populated by
localeconv(3) and localeconv_l(3). When setlocale(3) updates the global
locale, the lconv needs to be (lazily) reinitialized. To signal this,
we set flag variables in the locale structure. There are two problems:

  • The flags are set before the locale is fully updated, so a concurrent localeconv() call can observe partially initialized locale data.
  • No barriers ensure that localeconv() observes a fully initialized locale if a flag is set.

So, move the flag update appropriately, and use acq/rel barriers to
provide some synchronization. Note that this is inadequate in the face
of multiple concurrent calls to setlocale(3), but this is not expected
to work regardless.

Thanks to Henry Hu <henry.hu.sh@gmail.com> for providing a test case
demonstrating the race.

PR: 258360
Sponsored by: The FreeBSD Foundation

(cherry picked from commit 7eb138a9e53636366e615bdf04062fedc044bcea)

Details

Provenance
markjAuthored on Sep 17 2021, 2:44 PM
Parents
rG5aebce3790cc: rtld direct exec: add -d option
Branches
Unknown
Tags
Unknown
Reverted By
rG589daa05be6f: Revert "libc/locale: Fix races between localeconv(3) and setlocale(3)"