Messages in this thread Patch in this message | | | Subject | [PATCH] SELinux: Leak in error path | From | Alexander Nyberg <> | Date | Tue, 01 Mar 2005 01:32:52 +0100 |
| |
There's a leak here in the first error path.
Found by the Coverity tool.
Signed-off-by: Alexander Nyberg <alexn@dsv.su.se>
===== security/selinux/ss/conditional.c 1.3 vs edited ===== --- 1.3/security/selinux/ss/conditional.c 2005-01-05 03:48:22 +01:00 +++ edited/security/selinux/ss/conditional.c 2005-02-23 21:22:25 +01:00 @@ -401,8 +401,10 @@ static int cond_read_node(struct policyd expr->expr_type = le32_to_cpu(buf[0]); expr->bool = le32_to_cpu(buf[1]); - if (!expr_isvalid(p, expr)) + if (!expr_isvalid(p, expr)) { + kfree(expr); goto err; + } if (i == 0) { node->expr = expr;
- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
| |