[PATCH 59/63] policycoreutils: setfiles: fix use before initialized

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1


   This patch looks good to me. acked.

-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.11 (GNU/Linux)
Comment: Using GnuPG with Mozilla - http://enigmail.mozdev.org/

iEYEARECAAYFAk6wTTwACgkQrlYvE4MpobOHkACgzA83IE7Rd2jGbe39XK8jAzFy
R9EAoJex9vXf9gO7PD2BQ+qPlRAtxXcg
=K6t+
-----END PGP SIGNATURE-----
>From 879a0471bbec0dd67e1e6f282fb2b08cc7ba33a3 Mon Sep 17 00:00:00 2001
From: Dan Walsh <dwalsh@xxxxxxxxxx>
Date: Thu, 27 Oct 2011 10:38:27 -0400
Subject: [PATCH 59/63] policycoreutils: setfiles: fix use before initialized

There are code paths where ret can be returned without being initialized

Signed-off-by: Eric Paris <eparis@xxxxxxxxxx>
---
 policycoreutils/setfiles/restore.c |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/policycoreutils/setfiles/restore.c b/policycoreutils/setfiles/restore.c
index 9713345..6e1cbaa 100644
--- a/policycoreutils/setfiles/restore.c
+++ b/policycoreutils/setfiles/restore.c
@@ -108,7 +108,7 @@ static int match(const char *name, struct stat *sb, char **con)
 static int restore(FTSENT *ftsent)
 {
 	char *my_file = strdupa(ftsent->fts_path);
-	int ret;
+	int ret = -1;
 	security_context_t curcon = NULL, newcon = NULL;
 
 	if (match(my_file, ftsent->fts_statp, &newcon) < 0)
-- 
1.7.7


[Index of Archives]     [Selinux Refpolicy]     [Linux SGX]     [Fedora Users]     [Fedora Desktop]     [Yosemite Photos]     [Yosemite Camping]     [Yosemite Campsites]     [KDE Users]     [Gnome Users]

  Powered by Linux