bin/41887: wrong mode selected in ipf program for hash-entries

[Wolfgang.Stukenbrock%nagler-company.com@localhost]

>Number:         41887
>Category:       bin
>Synopsis:       wrong mode selected in ipf program for hash-entries
>Confidential:   no
>Severity:       serious
>Priority:       high
>Responsible:    bin-bug-people
>State:          open
>Class:          sw-bug
>Submitter-Id:   net
>Arrival-Date:   Fri Aug 14 08:15:00 +0000 2009
>Originator:     Wolfgang Stukenbrock
>Release:        NetBSD 4.0
>Organization:
Dr. Nagler & Company GmbH
        
>Environment:
        
        
System: NetBSD s012 4.0 NetBSD 4.0 (NSW-S012) #9: Fri Mar 13 12:31:52 CET 2009 
wgstuken@s012:/usr/src/sys/arch/amd64/compile/NSW-S012 amd64
Architecture: x86_64
Machine: amd64
>Description:
        In /usr/src/dist/ipf/tools/ipf_y.y while parsing an "addr" the variable 
"pooled" is set instead of "hashed" in the 5.th variant
        of an "addr". This looks like a copy-paste error. In all other cases 
the variable name correspond to the keyword just parsed.
        I'm not shure about the effect of this - we have found it while 
searching other problems in the ippoool(5) implementation.
>How-To-Repeat:
        found bey a look into the source code - you may define a "hash / 
YY_STR" line - the system should get into problems
>Fix:
        The following patch should be applied to /usr/src/dist/ipf/tools/ipf_y.y


--- ipf_y.y     2009/08/14 07:54:12     1.1
+++ ipf_y.y     2009/08/14 07:54:52
@@ -860,7 +860,7 @@
                                          $$.a.iplookuptype = IPLT_HASH;
                                          $$.a.iplookupsubtype = 0;
                                          $$.a.iplookupnum = $3; }
-       | hash '/' YY_STR               { pooled = 1;
+       | hash '/' YY_STR               { hashed = 1;
                                          $$.a.iplookuptype = IPLT_HASH;
                                          $$.a.iplookupsubtype = 1;
                                          strncpy($$.a.iplookupname, $3,

>Unformatted: