Re: [PATCH] MINOR: server: missing chunck allocation in srv_update_status()

2017-10-24 Thread Willy Tarreau
On Tue, Oct 24, 2017 at 07:12:15PM +0200, Olivier Houchard wrote: > On Tue, Oct 24, 2017 at 05:37:42PM +0200, Baptiste wrote: > > Hi, > > > > While testing Christopher's DNS "thread-safe" code, I found a bug in > > srv_update_status following a recent update (related to threads too). > > > > The

Re: [PATCH] MINOR: server: missing chunck allocation in srv_update_status()

2017-10-24 Thread Olivier Houchard
On Tue, Oct 24, 2017 at 07:12:15PM +0200, Olivier Houchard wrote: > On Tue, Oct 24, 2017 at 05:37:42PM +0200, Baptiste wrote: > > Hi, > > > > While testing Christopher's DNS "thread-safe" code, I found a bug in > > srv_update_status following a recent update (related to threads too). > > > > The

Re: [PATCH] MINOR: server: missing chunck allocation in srv_update_status()

2017-10-24 Thread Olivier Houchard
On Tue, Oct 24, 2017 at 05:37:42PM +0200, Baptiste wrote: > Hi, > > While testing Christopher's DNS "thread-safe" code, I found a bug in > srv_update_status following a recent update (related to threads too). > > The patch is in attachment. Ah you beat me at it ! I ran in the exact same issue.

[PATCH] MINOR: server: missing chunck allocation in srv_update_status()

2017-10-24 Thread Baptiste
ue, 24 Oct 2017 17:23:16 +0200 Subject: [PATCH] MINOR: server: missing chunck allocation in srv_update_status() Patch 64cc49cf introduced a bug in srv_update_status(): use of an uninitiaized chunck pointer leading to a segfault This patch aims at fixing this issue. --- sr