Both look strange to me. Shouldn't that be an ==

Regards
Henrik

On tor, 2007-09-27 at 11:28 +0200, Rafael Martinez (Squid development)
wrote:
> Amos, a little typo:
> 
> Acording to original semantics of safe_inet_addr, I think the conditioal
> expresion must be positive
> 
> SHOULD BE :
> 
> >      /* dotted notation */
> >      if (mask = asc)
> >           return true;
> > 
> 
> Otherwise, acl all src 0.0.0.0 fails to parse....
> Up to you the fix.
> 
> > Index: ACLIP.cc
> > ===================================================================
> > RCS file: /cvsroot/squid/squid3/src/ACLIP.cc,v
> > retrieving revision 1.8.2.45
> > retrieving revision 1.8.2.46
> > diff -C2 -d -r1.8.2.45 -r1.8.2.46
> > *** ACLIP.cc        21 Sep 2007 12:34:14 -0000      1.8.2.45
> > --- ACLIP.cc        24 Sep 2007 02:15:47 -0000      1.8.2.46
> > ***************
> > *** 205,209 ****
> >   
> >       /* dotted notation */
> > !     if (safe_inet_addr(asc, mask))
> >           return true;
> >   
> > --- 205,209 ----
> >   
> >       /* dotted notation */
> > !     if (!(mask = asc))
> >           return true;
> >   
> > 

Attachment: signature.asc
Description: This is a digitally signed message part

Reply via email to