On Thu, Jan 20, 2011 at 10:44:03AM +0100, Andreas Kohn wrote: > On Wed, 2011-01-19 at 20:38 +0100, Andreas Kohn wrote: > > Hi, > > > > while working further on the changes to allow for dynamic > > adding/removing of backend servers we noticed a potential problem: the > > path given for the 'stats socket' global option may get truncated when > > copying it into the sockaddr_un.sun_path field. > > > > Attached patch checks the length, and reports an error if truncation > > would happen. > > > > After some more digging around in the code I found standard.c:str2sun(), > which does check the length. So in the interest of avoiding duplication > attached a slightly different patch with the same goal.
Patch applied to both 1.4 and 1.5. Thanks Andreas! Willy