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 <[email protected]>
---
 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
[email protected]
https://lists.sourceforge.net/lists/listinfo/linuxwacom-devel

Reply via email to