-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1
On 07/07/2014 05:10 AM, Siddhesh Poyarekar wrote:
On Sat, Jul 05, 2014 at 10:13:08AM -0400, Carlos O'Donell wrote:
> The question of corrupt DSOs and hardening against them is entirely
> up to our good judgement. IIRC this patch was to harden against
> these DSOs. If we think there is a real bug we should file an issue
> and investigate.
Hardening against corrupt DSOs is a valid case, but the problem here
is that we don't *really* know why the fix works. In fact, I suspect
it only papers over the crashes and for any use that enables debug,
the crash will resurface. If it doesn't, then again I suspect that
the problem may be something else altogether whose symptom got fixed
somehow with this patch. Either way, there's nothing to indicate why
the fix works and why it is correct.
> Your call.
I'll revert this patch. I don't have nvidia hardware to see if the
crash still persists but if it does (that is someone reports it), then
we can take a look at it afresh and figure out the best possible fix,
this time with better documentation of what we're fixing than
"Horrible workaround for horribly broken software".
Sounds good to me. It seems negligent on our part to have patches in glibc
with poor rationale for their inclusion.
Cheers,
Carlos.
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1
Comment: Using GnuPG with Thunderbird -
http://www.enigmail.net/
iQEcBAEBAgAGBQJTuqM1AAoJECXvCkNsKkr/xVoH/Rkb8Em+lnpX/jnT5wSbY1Qb
m+uwPcAtprGoAgqjVMY9FPLe95zeXMyjeIpB+CnQZFlXZBXy2BxrSZhRsi9QFmRt
6PsK6OTJQNTAEsqcAprLgMIxLCFyZRF/gbuUCQAoZ+gPhyC4NtC0yBH3KgLpJSjQ
U3c7a9cuO8ozxPB/eTIj4bOVa7NuOaz7mYnntGpqtu2Lzfmtr7AZksb+X9E25t0x
dvvU6WUP7lbac6gg4X85eDOKWVNxsi+wDMF0HM++WDmPKJHFtJ3Kq5N134NUHY4z
7zW9me9ZaE0QPWhb/NLh4099XiFfZxppSve2FBy1TEeZbXoATNyx1JSFP2uK8g0=
=r24L
-----END PGP SIGNATURE-----