iio/iio.c: Add fix for access of variable before error check
Signed-off-by: Cosmin Popescu <gabrielcosmin.popescu@gmail.com> Signed-off-by: Mihai Tudor Panu <mihai.tudor.panu@intel.com>
This commit is contained in:
committed by
Mihai Tudor Panu
parent
2bb200aa04
commit
9cb508f5d2
@@ -131,7 +131,6 @@ mraa_iio_get_channel_data(mraa_iio_context dev)
|
|||||||
read(fd, readbuf, 31 * sizeof(char));
|
read(fd, readbuf, 31 * sizeof(char));
|
||||||
ret = sscanf(readbuf, "%ce:%c%u/%u>>%u", &shortbuf, &signchar, &chan->bits_used,
|
ret = sscanf(readbuf, "%ce:%c%u/%u>>%u", &shortbuf, &signchar, &chan->bits_used,
|
||||||
&padint, &chan->shift);
|
&padint, &chan->shift);
|
||||||
chan->bytes = padint / 8;
|
|
||||||
// probably should be 5?
|
// probably should be 5?
|
||||||
if (ret < 0) {
|
if (ret < 0) {
|
||||||
// cleanup
|
// cleanup
|
||||||
@@ -139,6 +138,7 @@ mraa_iio_get_channel_data(mraa_iio_context dev)
|
|||||||
close(fd);
|
close(fd);
|
||||||
return MRAA_IO_SETUP_FAILURE;
|
return MRAA_IO_SETUP_FAILURE;
|
||||||
}
|
}
|
||||||
|
chan->bytes = padint / 8;
|
||||||
chan->signedd = (signchar == 's');
|
chan->signedd = (signchar == 's');
|
||||||
chan->lendian = (shortbuf == 'l');
|
chan->lendian = (shortbuf == 'l');
|
||||||
if (chan->bits_used == 64) {
|
if (chan->bits_used == 64) {
|
||||||
|
|||||||
Reference in New Issue
Block a user