This is the mail archive of the libc-alpha@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] |
On 05/01/2016 01:17 AM, Anders Kaseorg wrote:
On Wed, 27 Apr 2016, Florian Weimer wrote:On 04/27/2016 05:36 PM, Anders Kaseorg wrote:Is this ready to be committed, or do you still have concerns?Yes please, this change looks fine.Do I need to do anything else? In case it wasnât clear, I donât have commit access.
I wasn't aware of that, sorry.I have looked at the patch again, and I think it is technically incorrect. The proper way to fix this would be to install a resolver callback which does nothing, and free the resolver state only if there is *no* callback (because we know we own it).
But we only ever use the thread-local resolver state anyway, so I'm just going to remove this code.
Florian
Index Nav: | [Date Index] [Subject Index] [Author Index] [Thread Index] | |
---|---|---|
Message Nav: | [Date Prev] [Date Next] | [Thread Prev] [Thread Next] |