This is the mail archive of the glibc-bugs@sourceware.org mailing list for the glibc project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

[Bug locale/22062] Unnecessary locking when reading configuration in iconv/gconv_conf.c


https://sourceware.org/bugzilla/show_bug.cgi?id=22062

Arjun Shankar <arjun.is at lostca dot se> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
            Summary|Data race when reading      |Unnecessary locking when
                   |configuration in            |reading configuration in
                   |iconv/gconv_conf.c          |iconv/gconv_conf.c

--- Comment #1 from Arjun Shankar <arjun.is at lostca dot se> ---
The review [1] of the patch I wrote for this revealed that no locking is
necessary at all because the entire routine is called via __libc_once. Given
this, the partial locking (when writing config to its data structures) during
__gconv_get_path should be thrown away.

[1] https://sourceware.org/ml/libc-alpha/2017-10/msg00428.html

-- 
You are receiving this mail because:
You are on the CC list for the bug.

Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]