projects / cascardo / linux.git / commit
? search:
summary | shortlog | log | commit | commitdiff | tree
(parent: ba4dc61) | patch
Staging: ipack: returning a freed pointer
authorDan Carpenter <dan.carpenter@oracle.com>
Thu, 10 May 2012 15:18:36 +0000 (18:18 +0300)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Thu, 10 May 2012 16:50:11 +0000 (09:50 -0700)
commit8bdeeb26c57012ea19a3b8fe3aeaecff377ae07d
treeb23368aea71a229f28d40d20c1488267b9830c97tree | snapshot
parentba4dc61fe8c545a5d6a68b63616776556b771f51commit | diff
Staging: ipack: returning a freed pointer

If ipack_device_register() returns an error, then we returned a freed
pointer.  The caller doesn't use it, but it means we return success to
the user instead of returning an error code.

I kind of rewrote the error handling in this function as a cleanup.

Cc: Samuel Iglesias Gonsalvez <siglesias@igalia.com>
Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/staging/ipack/bridges/tpci200.c diff | blob | history
Unnamed repository; edit this file 'description' to name the repository.