Created attachment 27801 [details]
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.