There is no need to free the 'tool' variable, since the "error" label is
only ever reached if 'tool' is already null.

V575 The null pointer is passed into 'free' function. Inspect the first 
argument.

Signed-off-by: Jason Gerecke <jason.gere...@wacom.com>
---
 src/wcmConfig.c | 1 -
 1 file changed, 1 deletion(-)

diff --git a/src/wcmConfig.c b/src/wcmConfig.c
index f85f1ca..69310e9 100644
--- a/src/wcmConfig.c
+++ b/src/wcmConfig.c
@@ -106,7 +106,6 @@ static int wcmAllocate(InputInfoPtr pInfo)
        return 1;
 
 error:
-       free(tool);
        wcmFreeCommon(&common);
        free(priv);
        return 0;
-- 
2.18.0


------------------------------------------------------------------------------
Check out the vibrant tech community on one of the world's most
engaging tech sites, Slashdot.org! http://sdm.link/slashdot
_______________________________________________
Linuxwacom-devel mailing list
Linuxwacom-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/linuxwacom-devel

Reply via email to