[BusyBox 0000903]: Missing #includes in some applets

bugs at busybox.net bugs at busybox.net
Sun Sep 3 14:53:24 UTC 2006


The following issue has been CLOSED 
====================================================================== 
http://busybox.net/bugs/view.php?id=903 
====================================================================== 
Reported By:                iggarpe
Assigned To:                BusyBox
====================================================================== 
Project:                    BusyBox
Issue ID:                   903
Category:                   Standards Compliance
Reproducibility:            always
Severity:                   minor
Priority:                   normal
Status:                     closed
Resolution:                 open
Fixed in Version:           
====================================================================== 
Date Submitted:             06-14-2006 02:22 PDT
Last Modified:              09-03-2006 07:53 PDT
====================================================================== 
Summary:                    Missing #includes in some applets
Description: 
insmod.c    should #include "libgen.h"
arping.c    should #include "net/if_arp.h"
ifconfig.c  should #include "sys/types.h"
nslookup.c  should #include "resolv.h"

And wherever "sys/fcntl.h" is used, "fcntl.h" should be used instead.

I say "should" because those files use functions and structures which are
defined in those headers. Currently, busybox WILL compile without errors
in any standard unix box because those headers are included somewhere in
the other included files.

However, if one tries to compile in other architechtures with different
libc, the erros will surface.

====================================================================== 

---------------------------------------------------------------------- 
 landley - 06-28-06 07:38  
---------------------------------------------------------------------- 
Which architectures?

I'm not very interested in "fixing" purely theoretical errors... 

---------------------------------------------------------------------- 
 iggarpe - 06-28-06 09:42  
---------------------------------------------------------------------- 
The mentioned files do use structures and functions that are defined in
those includes, so the includes should be there. Or so the man pages say.

I think this should be fixed for correctness' sake, AND because it's free:
the fix is trivial (the hard part, identifying those missing includes is
already done). 

---------------------------------------------------------------------- 
 landley - 06-28-06 13:03  
---------------------------------------------------------------------- 
So no architectures are impacted by this, and this is a purely theoretical
error.

In 1.3 I'm moving most of the #includes to libbb.h, and out of the
individual applets.  Until then, my lack of caring about this is just
about absolute.  During 1.3, my lack of caring about this will merely be
monumental. 

---------------------------------------------------------------------- 
 vda - 09-03-06 07:53  
---------------------------------------------------------------------- 
Mostly theoretical problem. A patch is missing too. 

Issue History 
Date Modified   Username       Field                    Change               
====================================================================== 
06-14-06 02:22  iggarpe        New Issue                                    
06-14-06 02:22  iggarpe        Status                   new => assigned     
06-14-06 02:22  iggarpe        Assigned To               => BusyBox         
06-28-06 07:38  landley        Note Added: 0001471                          
06-28-06 09:42  iggarpe        Note Added: 0001472                          
06-28-06 13:03  landley        Note Added: 0001473                          
09-03-06 07:53  vda            Status                   assigned => closed  
09-03-06 07:53  vda            Note Added: 0001611                          
======================================================================




More information about the busybox-cvs mailing list