Upon error the 'n' string was never freed.
Found by coverity.
Signed-off-by: Hannes Reinecke <[email protected]>
---
libmultipath/structs.c | 4 +++-
1 file changed, 3 insertions(+), 1 deletion(-)
diff --git a/libmultipath/structs.c b/libmultipath/structs.c
index 502d98e..ed62b07 100644
--- a/libmultipath/structs.c
+++ b/libmultipath/structs.c
@@ -653,9 +653,11 @@ remove_feature(char **f, char *o)
* about to be removed
*/
p = strchr(*f, ' ');
- if (!p)
+ if (!p) {
/* Internal error, feature string inconsistent */
+ FREE(n);
return 1;
+ }
while (*p == ' ')
p++;
p--;
--
2.6.6
--
dm-devel mailing list
[email protected]
https://www.redhat.com/mailman/listinfo/dm-devel