media: drxj: better handle errors

as reported by smatch:
	drivers/media/dvb-frontends/drx39xyj/drxj.c:2157 drxj_dap_atomic_read_write_block() error: uninitialized symbol 'word'.

The driver doesn't check if a read error occurred. Add such
check.

Signed-off-by: Mauro Carvalho Chehab <mchehab@s-opensource.com>

diff --git a/drivers/media/dvb-frontends/drx39xyj/drxj.c b/drivers/media/dvb-frontends/drx39xyj/drxj.c
index 8cbd8cc21059d8a94112608170e386ebcf7866d5..77d3a44177ba0d2230073d3836dd36fa1ca0d9f7 100644 (file)
--- a/drivers/media/dvb-frontends/drx39xyj/drxj.c
+++ b/drivers/media/dvb-frontends/drx39xyj/drxj.c
@@ -2151,9 +2151,13 @@ int drxj_dap_atomic_read_write_block(struct i2c_device_addr *dev_addr,
        if (read_flag) {
                /* read data from buffer */
                for (i = 0; i < (datasize / 2); i++) {
-                       drxj_dap_read_reg16(dev_addr,
-                                           (DRXJ_HI_ATOMIC_BUF_START + i),
-                                          &word, 0);
+                       rc = drxj_dap_read_reg16(dev_addr,
+                                                (DRXJ_HI_ATOMIC_BUF_START + i),
+                                                &word, 0);
+                       if (rc) {
+                               pr_err("error %d\n", rc);
+                               goto rw_error;
+                       }
                        data[2 * i] = (u8) (word & 0xFF);
                        data[(2 * i) + 1] = (u8) (word >> 8);
                }