diff options
| author | James Bottomley <James.Bottomley@suse.de> | 2009-11-03 19:33:07 +0100 |
|---|---|---|
| committer | James Bottomley <James.Bottomley@suse.de> | 2010-01-18 17:48:05 +0100 |
| commit | e3deec090558d5cb5ffdc574e5560f3ed9723394 (patch) | |
| tree | c76a5e26a3e08598ada0a2de34adcdf714aa7168 /include/scsi | |
| parent | [SCSI] aic79xx: check for non-NULL scb in ahd_handle_nonpkt_busfree (diff) | |
| download | linux-e3deec090558d5cb5ffdc574e5560f3ed9723394.tar.xz linux-e3deec090558d5cb5ffdc574e5560f3ed9723394.zip | |
[SCSI] eliminate potential kmalloc failure in scsi_get_vpd_page()
The best way to fix this is to eliminate the intenal kmalloc() and
make the caller allocate the required amount of storage.
Signed-off-by: James Bottomley <James.Bottomley@suse.de>
Diffstat (limited to 'include/scsi')
| -rw-r--r-- | include/scsi/scsi_device.h | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/include/scsi/scsi_device.h b/include/scsi/scsi_device.h index 7c4449900c24..d80b6dbed1ca 100644 --- a/include/scsi/scsi_device.h +++ b/include/scsi/scsi_device.h @@ -348,7 +348,8 @@ extern int scsi_mode_select(struct scsi_device *sdev, int pf, int sp, struct scsi_sense_hdr *); extern int scsi_test_unit_ready(struct scsi_device *sdev, int timeout, int retries, struct scsi_sense_hdr *sshdr); -extern unsigned char *scsi_get_vpd_page(struct scsi_device *, u8 page); +extern int scsi_get_vpd_page(struct scsi_device *, u8 page, unsigned char *buf, + int buf_len); extern int scsi_device_set_state(struct scsi_device *sdev, enum scsi_device_state state); extern struct scsi_event *sdev_evt_alloc(enum scsi_device_event evt_type, |