On 05/03/2010 02:04 PM, Amos Jeffries wrote: > Thanks for the thought, but... > > Code in Squid is NOT permitted to include system headers before the > FD_* compat code. Kernel defines will be allocated with incompatible > size and overflows happen. > > Can you provide a full compiler trace of the clash so we can fix the > include sequence? > > Amos Hello Amos, Here i have another patch fixing the include file ordering for the ntlm_auth helper. Not sure if this is the best patch but it works for me. -- Matthias
Fix include order to ensure that FD_SETSIZE from the compat/fdsetsize.h is set before it is set by sys/select.h (included by stdlib.h). --- helpers/ntlm_auth/smb_lm/smbval/rfcnb-util.c.orig Mon May 3 16:17:35 2010 +++ helpers/ntlm_auth/smb_lm/smbval/rfcnb-util.c Mon May 3 16:21:23 2010 @@ -23,12 +23,11 @@ * Foundation, Inc., 675 Mass Ave, Cambridge, MA 02139, USA. */ -#include <string.h> -#include <stdlib.h> - #include "std-includes.h" #include "rfcnb-priv.h" #include "rfcnb-util.h" +#include <string.h> +#include <stdlib.h> #include "rfcnb-io.h" #include <arpa/inet.h> --- helpers/ntlm_auth/smb_lm/smbval/session.c.orig Sun May 2 12:47:07 2010 +++ helpers/ntlm_auth/smb_lm/smbval/session.c Mon May 3 16:20:53 2010 @@ -23,9 +23,6 @@ * Foundation, Inc., 675 Mass Ave, Cambridge, MA 02139, USA. */ -#include <string.h> -#include <stdlib.h> - int RFCNB_errno = 0; int RFCNB_saved_errno = 0; #define RFCNB_ERRNO @@ -34,6 +31,8 @@ #include <netinet/tcp.h> #include "rfcnb-priv.h" #include "rfcnb-util.h" +#include <string.h> +#include <stdlib.h> #include "rfcnb-io.h" #include "rfcnb.h"
