diff options
| author |   Figo.zhang <figo1802@gmail.com> | 2009-06-06 19:11:31 +0800 |
|---|---|---|
| committer |   Greg Kroah-Hartman <gregkh@suse.de> | 2009-09-15 12:01:33 -0700 |
| commit | b455073c885cf78cfea1704ce9587806ec8e3c30 (patch) | |
| tree | ecce71c4119e24a960204fa4673bf7521758520e /drivers/staging/comedi | |
| parent | Staging: Comedi: Lindent changes to comdi driver in staging tree (diff) | |
| download | linux-dev-b455073c885cf78cfea1704ce9587806ec8e3c30.tar.xz linux-dev-b455073c885cf78cfea1704ce9587806ec8e3c30.zip | |
Staging: comedi: no need for checking vfree call
vfree() does it's own NULL checking,so no need for check before
calling it.
'pages' is local variable argment,so in v2, it is no need assignment
to NULL.
Signed-off-by: Figo.zhang <figo1802@gmail.com>
Acked-by: Pekka Enberg <penberg@cs.helsinki.fi>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
Diffstat (limited to 'drivers/staging/comedi')
| -rw-r--r-- | drivers/staging/comedi/drivers.c | 5 |
1 files changed, 2 insertions, 3 deletions
diff --git a/drivers/staging/comedi/drivers.c b/drivers/staging/comedi/drivers.c index 5157c89ec4ee..dc53aeeac68f 100644 --- a/drivers/staging/comedi/drivers.c +++ b/drivers/staging/comedi/drivers.c @@ -509,9 +509,8 @@ int comedi_buf_alloc(struct comedi_device *dev, struct comedi_subdevice *s, async->prealloc_buf = vmap(pages, n_pages, VM_MAP, PAGE_KERNEL_NOCACHE); } - if (pages) { - vfree(pages); - } + vfree(pages); + if (async->prealloc_buf == NULL) { /* Some allocation failed above. */ if (async->buf_page_list) { |