[BusyBox 0000428]: Extra space character (0x20) is added to last string option

[bugs at busybox.net]

The following issue has been SUBMITTED. 
====================================================================== 
http://busybox.net/bugs/view.php?id=428 
====================================================================== 
Reported By:                mareksk
Assigned To:                BusyBox
====================================================================== 
Project:                    BusyBox
Issue ID:                   428
Category:                   Kernel Module Support
Reproducibility:            always
Severity:                   major
Priority:                   normal
Status:                     assigned
====================================================================== 
Date Submitted:             09-13-2005 10:55 PDT
Last Modified:              09-13-2005 10:55 PDT
====================================================================== 
Summary:                    Extra space character (0x20) is added to last string
option
Description: 
Extra space character (0x20) is always being added to the last option
passed to insmod. In case of the last option is a string value, this value
is expanded by one character. For instance, if the last option is arg=1234,
the string passed to a module is "1234 ".

The code below points a part of 'mod_insert' function, where a simple fix
could be added

for ( i = 0; i < argc; i++ ) {
  strcat ( head-> m_options, argv [i] );
  /* this prevents from adding an extra 0x20 after the last option */
  if( (i+1) < argc ) {  
    strcat ( head-> m_options, " " );
  }
}

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

Issue History 
Date Modified   Username       Field                    Change               
====================================================================== 
09-13-05 10:55  mareksk        New Issue                                    
09-13-05 10:55  mareksk        Status                   new => assigned     
09-13-05 10:55  mareksk        Assigned To               => BusyBox         
======================================================================