diff options
author | H Hartley Sweeten <hsweeten@visionengravers.com> | 2013-04-12 16:58:45 (GMT) |
---|---|---|
committer | Greg Kroah-Hartman <gregkh@linuxfoundation.org> | 2013-04-12 22:09:59 (GMT) |
commit | dd671a39481eaf8ebd512ea8cba25d62066d078f (patch) | |
tree | d05820a244728cdb972dc2949752aac6063f44b1 /drivers/staging | |
parent | 53510352587c5350a8ae0c150a73244ce176ae40 (diff) | |
download | linux-fsl-qoriq-dd671a39481eaf8ebd512ea8cba25d62066d078f.tar.xz |
staging: comedi: unionxx5: allocate 'usp' before using it
As reported by the kbuild test robot, the 'usp' pointer needs to be
allocated before being used.
Signed-off-by: H Hartley Sweeten <hsweeten@visionengravers.com>
Reported-by: kbuild test robot <fengguang.wu@intel.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Diffstat (limited to 'drivers/staging')
-rw-r--r-- | drivers/staging/comedi/drivers/unioxx5.c | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/drivers/staging/comedi/drivers/unioxx5.c b/drivers/staging/comedi/drivers/unioxx5.c index 6661cf7..1b19cc2 100644 --- a/drivers/staging/comedi/drivers/unioxx5.c +++ b/drivers/staging/comedi/drivers/unioxx5.c @@ -375,15 +375,15 @@ static int __unioxx5_subdev_init(struct comedi_device *dev, int i, to, ndef_flag = 0; int ret; + usp = kzalloc(sizeof(*usp), GFP_KERNEL); + if (usp == NULL) + return -ENOMEM; + ret = __comedi_request_region(dev, iobase, UNIOXX5_SIZE); if (ret) return ret; usp->usp_iobase = iobase; - usp = kzalloc(sizeof(*usp), GFP_KERNEL); - if (usp == NULL) - return -ENOMEM; - /* defining modules types */ for (i = 0; i < 12; i++) { to = 10000; |