New: Security problem in the FTP URL for obtaining QLogic Fibre Channel HBA firmware

https://bugzilla.kernel.org/show_bug.cgi?id=16901

Summary: Security problem in the FTP URL for obtaining QLogic
Fibre Channel HBA firmware
Product: SCSI Drivers
Version: 2.5
Kernel Version: 2.6.35.3
Platform: All
OS/Version: Linux
Tree: Mainline
Status: NEW
Severity: high
Priority: P1
Component: QLOGIC QLA2XXX
AssignedTo: [email protected]
ReportedBy: [email protected]
Regression: No

Created an attachment (id=27801)
--> (https://bugzilla.kernel.org/attachment.cgi?id=27801)
Proof that firmware files can be replaces with malicious ones

The qla2xxx driver in the latest stable 2.6.35.3 prints dangerous URL (ftp.qlogic.com) for obtaining new firmware for their adapters.

From: linux-2.6.35.3/drivers/scsi/qla2xxx/qla_init.c ---- quote -----
#define QLA_FW_URL "ftp://ftp.qlogic.com/outgoing/linux/firmware/"

int
qla2x00_load_risc(scsi_qla_host_t *vha, uint32_t *srisc_addr)

{
        int     rval;
        int     i, fragment;
        uint16_t *wcode, *fwcode;
        uint32_t risc_addr, risc_size, fwclen, wlen, *seg;
        struct fw_blob *blob;
        struct qla_hw_data *ha = vha->hw;
        struct req_que *req = ha->req_q_map[0];

        /* Load firmware blob. */
        blob = qla2x00_request_firmware(vha);
        if (!blob) {
                qla_printk(KERN_ERR, ha, "Firmware image unavailable.\n");
                qla_printk(KERN_ERR, ha, "Firmware images can be retrieved "
                    "from: " QLA_FW_URL ".\n");
                return QLA_FUNCTION_FAILED;
        }

---- end quote -----

It is dangerous to give the users this URL for obtaining new firmware images,
because this QLogic FTP site isn't secure at all and everyone anonymously can exchange the original firmware .bin files with their own files. The outgoing/linux/firmware folder is read-only and files cannot be replaced, or created new ones. The folder itself cannot be renamed too, but everyone can rename linux folder to linux2 for example, then create a new linux folder with firmware inside and put their own files instead of the 'official' ones from QLogic.
It can be done easy and you can see the attached screenshot for proof.
Until better security is implemented on this FTP (or better place for storing firmware releases is found) I recommend to print a warning to all users of this driver about security problems with the QLogic FTP site.

Configure bugmail: https://bugzilla.kernel.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are watching the assignee of the bug.

https://bugzilla.kernel.org/show_bug.cgi?id=16901

Andrew Vasquez <[email protected]> changed:

What |Removed |Added
----------------------------------------------------------------------------
CC| |[email protected]

--- Comment #1 from Andrew Vasquez <[email protected]> 2010-08-24 16:20:56 --- Thank you for bringing this to our attention. We've notified the IT group to have these permission problems with the files and directories addressed.

Configure bugmail: https://bugzilla.kernel.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are watching the assignee of the bug.

https://bugzilla.kernel.org/show_bug.cgi?id=16901

Alan <[email protected]> changed:

What |Removed |Added
----------------------------------------------------------------------------

             Status|NEW                         |RESOLVED
                 CC|                            |[email protected]
         Resolution|                            |OBSOLETE

Configure bugmail: https://bugzilla.kernel.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are watching the assignee of the bug.

https://bugzilla.kernel.org/show_bug.cgi?id=16901

Alan <[email protected]> changed:

What |Removed |Added
----------------------------------------------------------------------------
Status|RESOLVED |CLOSED

Configure bugmail: https://bugzilla.kernel.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are watching the assignee of the bug.