• Arnd Bergmann's avatar
    scsi: myrb: fix sprintf buffer overflow warning · f8f4adc1
    Arnd Bergmann authored
    gcc warns that the 12 byte fw_version field might not be long enough to
    contain the generated firmware name string:
    
    drivers/scsi/myrb.c: In function 'myrb_get_hba_config':
    drivers/scsi/myrb.c:1052:38: error: '%02d' directive writing between 2 and 3 bytes into a region of size between 2 and 5 [-Werror=format-overflow=]
      sprintf(cb->fw_version, "%d.%02d-%c-%02d",
                                          ^~~~
    drivers/scsi/myrb.c:1052:26: note: directive argument in the range [0, 255]
      sprintf(cb->fw_version, "%d.%02d-%c-%02d",
                              ^~~~~~~~~~~~~~~~~
    drivers/scsi/myrb.c:1052:2: note: 'sprintf' output between 10 and 14 bytes into a destination of size 12
      sprintf(cb->fw_version, "%d.%02d-%c-%02d",
      ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
       enquiry2->fw.major_version,
       ~~~~~~~~~~~~~~~~~~~~~~~~~~~
       enquiry2->fw.minor_version,
       ~~~~~~~~~~~~~~~~~~~~~~~~~~~
       enquiry2->fw.firmware_type,
       ~~~~~~~~~~~~~~~~~~~~~~~~~~~
       enquiry2->fw.turn_id);
       ~~~~~~~~~~~~~~~~~~~~~
    
    I have not checked whether there are appropriate range checks before the
    sprintf, but there is a range check after it that will bail out in case
    of out of range version numbers. This means we can simply use snprintf()
    instead of sprintf() to limit the output buffer size, and it will work
    correctly.
    
    Fixes: 081ff398 ("scsi: myrb: Add Mylex RAID controller (block interface)")
    Signed-off-by: default avatarArnd Bergmann <arnd@arndb.de>
    Reviewed-by: default avatarHannes Reinecke <hare@suse.com>
    Signed-off-by: default avatarMartin K. Petersen <martin.petersen@oracle.com>
    f8f4adc1
myrb.c 97.2 KB