diff options
| author | Linus Torvalds <torvalds@linux-foundation.org> | 2008-12-22 17:40:48 -0500 |
|---|---|---|
| committer | Linus Torvalds <torvalds@linux-foundation.org> | 2008-12-22 17:40:48 -0500 |
| commit | 0099f77e0a224cc461d0d2930ef0fdb32c61ba64 (patch) | |
| tree | c4f6252067f6d581779eba4039020279ce11c5f3 | |
| parent | 2e8569669805c2d2620527464c395279b74892fc (diff) | |
| parent | 9ecab6e5bf87f96dc2fa89cc9e8d5576fbde4325 (diff) | |
Merge git://git.kernel.org/pub/scm/linux/kernel/git/bart/ide-2.6
* git://git.kernel.org/pub/scm/linux/kernel/git/bart/ide-2.6:
drivers/ide/{cs5530.c,sc1200.c}: Move a dereference below a NULL test
| -rw-r--r-- | drivers/ide/cs5530.c | 3 | ||||
| -rw-r--r-- | drivers/ide/sc1200.c | 3 |
2 files changed, 4 insertions, 2 deletions
diff --git a/drivers/ide/cs5530.c b/drivers/ide/cs5530.c index 53f079cc00af..d8ede85fe17f 100644 --- a/drivers/ide/cs5530.c +++ b/drivers/ide/cs5530.c | |||
| @@ -81,11 +81,12 @@ static u8 cs5530_udma_filter(ide_drive_t *drive) | |||
| 81 | { | 81 | { |
| 82 | ide_hwif_t *hwif = drive->hwif; | 82 | ide_hwif_t *hwif = drive->hwif; |
| 83 | ide_drive_t *mate = ide_get_pair_dev(drive); | 83 | ide_drive_t *mate = ide_get_pair_dev(drive); |
| 84 | u16 *mateid = mate->id; | 84 | u16 *mateid; |
| 85 | u8 mask = hwif->ultra_mask; | 85 | u8 mask = hwif->ultra_mask; |
| 86 | 86 | ||
| 87 | if (mate == NULL) | 87 | if (mate == NULL) |
| 88 | goto out; | 88 | goto out; |
| 89 | mateid = mate->id; | ||
| 89 | 90 | ||
| 90 | if (ata_id_has_dma(mateid) && __ide_dma_bad_drive(mate) == 0) { | 91 | if (ata_id_has_dma(mateid) && __ide_dma_bad_drive(mate) == 0) { |
| 91 | if ((mateid[ATA_ID_FIELD_VALID] & 4) && | 92 | if ((mateid[ATA_ID_FIELD_VALID] & 4) && |
diff --git a/drivers/ide/sc1200.c b/drivers/ide/sc1200.c index f1a8758e3a99..ec7f766ef5e4 100644 --- a/drivers/ide/sc1200.c +++ b/drivers/ide/sc1200.c | |||
| @@ -104,11 +104,12 @@ static u8 sc1200_udma_filter(ide_drive_t *drive) | |||
| 104 | { | 104 | { |
| 105 | ide_hwif_t *hwif = drive->hwif; | 105 | ide_hwif_t *hwif = drive->hwif; |
| 106 | ide_drive_t *mate = ide_get_pair_dev(drive); | 106 | ide_drive_t *mate = ide_get_pair_dev(drive); |
| 107 | u16 *mateid = mate->id; | 107 | u16 *mateid; |
| 108 | u8 mask = hwif->ultra_mask; | 108 | u8 mask = hwif->ultra_mask; |
| 109 | 109 | ||
| 110 | if (mate == NULL) | 110 | if (mate == NULL) |
| 111 | goto out; | 111 | goto out; |
| 112 | mateid = mate->id; | ||
| 112 | 113 | ||
| 113 | if (ata_id_has_dma(mateid) && __ide_dma_bad_drive(mate) == 0) { | 114 | if (ata_id_has_dma(mateid) && __ide_dma_bad_drive(mate) == 0) { |
| 114 | if ((mateid[ATA_ID_FIELD_VALID] & 4) && | 115 | if ((mateid[ATA_ID_FIELD_VALID] & 4) && |
