diff options
| author | Dan Carpenter <dan.carpenter@oracle.com> | 2016-12-01 08:48:30 +0300 |
|---|---|---|
| committer | David S. Miller <davem@davemloft.net> | 2016-12-06 12:18:22 -0500 |
| commit | 163117e8d4fd7a235ec48479e31bbda0c74eff56 (patch) | |
| tree | 3f5aa5e729a87e0b1fa9f78155442bbe675a75f9 /sound/sparc | |
| parent | 88abd8249ee8bcebb98c90e890ea5e342db832af (diff) | |
| download | lwn-163117e8d4fd7a235ec48479e31bbda0c74eff56.tar.gz lwn-163117e8d4fd7a235ec48479e31bbda0c74eff56.zip | |
dbri: move dereference after check for NULL
We accidentally introduced a dereference before the NULL check in
xmit_descs() as part of silencing a GCC warning.
Fixes: 16f46050e709 ("dbri: Fix compiler warning")
Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'sound/sparc')
| -rw-r--r-- | sound/sparc/dbri.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/sound/sparc/dbri.c b/sound/sparc/dbri.c index 3fe4468ea2c5..52063b262667 100644 --- a/sound/sparc/dbri.c +++ b/sound/sparc/dbri.c @@ -1702,7 +1702,7 @@ interrupts are disabled. static void xmit_descs(struct snd_dbri *dbri) { struct dbri_streaminfo *info; - u32 dvma_addr = (u32)dbri->dma_dvma; + u32 dvma_addr; s32 *cmd; unsigned long flags; int first_td; @@ -1710,6 +1710,7 @@ static void xmit_descs(struct snd_dbri *dbri) if (dbri == NULL) return; /* Disabled */ + dvma_addr = (u32)dbri->dma_dvma; info = &dbri->stream_info[DBRI_REC]; spin_lock_irqsave(&dbri->lock, flags); |