teknoraver / rpms / systemd

Forked from rpms/systemd 2 months ago
Clone

Blame 0006-udev-fix-invalid-free-in-enable_name_policy.patch

Zbigniew Jędrzejewski-Szmek 5509c0
From e6e0462204a04517c6f31b7ca631d715c6ab04f8 Mon Sep 17 00:00:00 2001
Zbigniew Jędrzejewski-Szmek 5509c0
From: Andreas Henriksson <andreas@fatal.se>
Zbigniew Jędrzejewski-Szmek 5509c0
Date: Fri, 13 Jun 2014 18:48:21 +0200
Zbigniew Jędrzejewski-Szmek 5509c0
Subject: [PATCH] udev: fix invalid free() in enable_name_policy()
Zbigniew Jędrzejewski-Szmek 5509c0
Zbigniew Jędrzejewski-Szmek 5509c0
static bool enable_name_policy(...) in ./src/udev/net/link-config.c
Zbigniew Jędrzejewski-Szmek 5509c0
calls proc_cmdline(...) to get "line" initialized, but
Zbigniew Jędrzejewski-Szmek 5509c0
proc_cmdline(...) does not guarantee that atleast when both
Zbigniew Jędrzejewski-Szmek 5509c0
conditions (detect_container(NULL) > 0) and
Zbigniew Jędrzejewski-Szmek 5509c0
read_full_file(...) returned < 0.
Zbigniew Jędrzejewski-Szmek 5509c0
Zbigniew Jędrzejewski-Szmek 5509c0
(cherry picked from commit f8a0bb5285024b6ce372c3157e761e6543ebdcd2)
Zbigniew Jędrzejewski-Szmek 5509c0
---
Zbigniew Jędrzejewski-Szmek 5509c0
 src/udev/net/link-config.c | 2 +-
Zbigniew Jędrzejewski-Szmek 5509c0
 1 file changed, 1 insertion(+), 1 deletion(-)
Zbigniew Jędrzejewski-Szmek 5509c0
Zbigniew Jędrzejewski-Szmek 5509c0
diff --git a/src/udev/net/link-config.c b/src/udev/net/link-config.c
Zbigniew Jędrzejewski-Szmek 5509c0
index a9acc3dd..7a9d01bc 100644
Zbigniew Jędrzejewski-Szmek 5509c0
--- a/src/udev/net/link-config.c
Zbigniew Jędrzejewski-Szmek 5509c0
+++ b/src/udev/net/link-config.c
Zbigniew Jędrzejewski-Szmek 5509c0
@@ -185,7 +185,7 @@ static int load_link(link_config_ctx *ctx, const char *filename) {
Zbigniew Jędrzejewski-Szmek 5509c0
 }
Zbigniew Jędrzejewski-Szmek 5509c0
 
Zbigniew Jędrzejewski-Szmek 5509c0
 static bool enable_name_policy(void) {
Zbigniew Jędrzejewski-Szmek 5509c0
-        _cleanup_free_ char *line;
Zbigniew Jędrzejewski-Szmek 5509c0
+        _cleanup_free_ char *line = NULL;
Zbigniew Jędrzejewski-Szmek 5509c0
         char *w, *state;
Zbigniew Jędrzejewski-Szmek 5509c0
         int r;
Zbigniew Jędrzejewski-Szmek 5509c0
         size_t l;