cvs commit: src/sys/netinet ip_fw_pfil.c ip_input.c

Julian Elischer julian at elischer.org
Sat Oct 21 00:10:13 UTC 2006


Sam Leffler wrote:
> Sam Leffler wrote:
>> Julian Elischer wrote:
>>> julian      2006-10-20 19:32:08 UTC
>>>
>>>   FreeBSD src repository
>>>
>>>   Modified files:
>>>     sys/netinet          ip_fw_pfil.c ip_input.c 
>>>   Log:
>>>   Move some variables to a more likely place
>>>   and remove "temporary" stuff that is not needed any more.
>>>   
>>>   Revision  Changes    Path
>>>   1.22      +8 -0      src/sys/netinet/ip_fw_pfil.c
>>> http://cvsweb.FreeBSD.org/src/sys/netinet/ip_fw_pfil.c.diff?r1=1.21&r2=1.22
>>>   1.321     +0 -12     src/sys/netinet/ip_input.c
>>> http://cvsweb.FreeBSD.org/src/sys/netinet/ip_input.c.diff?r1=1.320&r2=1.321
>>>
>>>
>> This change breaks kernels w/o PFIL_HOOKS.
> 
> I misspoke.  It requires ipfw or similar to build:
> 
> if_ethersubr.o(.text+0x828): In function `ether_output_frame':
> /usr/sam/avila/sys/net/if_ethersubr.c:386: undefined reference to
> `ip_fw_chk_ptr'
> if_ethersubr.o(.text+0xb20): In function `ether_ipfw_chk':
> /usr/sam/avila/sys/net/if_ethersubr.c:481: undefined reference to
> `fw_one_pass'
> if_ethersubr.o(.text+0xb28):/usr/sam/avila/sys/net/if_ethersubr.c:481:
> undefined reference to `ip_fw_chk_ptr'
> if_ethersubr.o(.text+0xb2c):/usr/sam/avila/sys/net/if_ethersubr.c:481:
> undefined reference to `ip_dn_io_ptr'
> if_ethersubr.o(.text+0x1274): In function `ether_demux':
> /usr/sam/avila/sys/net/if_ethersubr.c:830: undefined reference to
> `ip_fw_chk_ptr
> 
> That's because you moved the variable out of the ip code and into ipfw.
> 
> 	Sam
gotcha  sorry

I'll fix it..
I thought I tested all the combinations.. but I just figured out that
I moved it to the wrong place..
I should have put it into pfil.c not ip_fw_pfil.c
pfil.c is dependent on inet and ether. so should exist at all times that
ip_input.c did, but is more relevant.



(it's part of another change that I'm working on that may or may not
come into the FreeBSD tree).




More information about the cvs-all mailing list