RTOS/rtems
1
0
Fork 0
forked from Imagelibrary/rtems
Code Pull Requests Activity

leon,grtm: ioctl() shouldn't access pointer before validated

Browse Source 
This is a fix for a coverity warning
This commit is contained in:
Daniel Hellstrom
2017-02-03 08:53:35 +01:00
parent fec8288f51
commit cc8a50ab5c
1 changed files with 2 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

3
c/src/lib/libbsp/sparc/shared/tmtc/grtm.c
View File

@@ -1188,7 +1188,7 @@ static rtems_device_driver grtm_ioctl(rtems_device_major_number major, rtems_dev
struct grtm_priv *pDev; struct grtm_priv *pDev;
struct drvmgr_dev *dev; struct drvmgr_dev *dev;
rtems_libio_ioctl_args_t *ioarg = (rtems_libio_ioctl_args_t *)arg; rtems_libio_ioctl_args_t *ioarg = (rtems_libio_ioctl_args_t *)arg;
unsigned int *data = ioarg->buffer; unsigned int *data;
int status; int status;
struct grtm_ioc_config *cfg; struct grtm_ioc_config *cfg;
struct grtm_ioc_hw_status *hwregs; struct grtm_ioc_hw_status *hwregs;
@@ -1209,6 +1209,7 @@ static rtems_device_driver grtm_ioctl(rtems_device_major_number major, rtems_dev
if (!ioarg) if (!ioarg)
return RTEMS_INVALID_NAME; return RTEMS_INVALID_NAME;
data = ioarg->buffer;
ioarg->ioctl_return = 0; ioarg->ioctl_return = 0;
switch(ioarg->command) { switch(ioarg->command) {
case GRTM_IOC_START: case GRTM_IOC_START: