hal: Branch 'master'

Danny Kukawka dkukawka at kemper.freedesktop.org
Fri Apr 4 04:42:13 PDT 2008


 hald/hald_runner.c |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

New commits:
commit 89591f68dd545adaf5f5564b973077c73559eb79
Author: Danny Kukawka <danny.kukawka at web.de>
Date:   Fri Apr 4 13:42:07 2008 +0200

    fixed check for return value of uname()
    
    Fixed check for return value of uname() as pointed out by Phi Tran
    <Phi.Tran at sun.com> on the HAL mailing list:
    
    > hald_runner.c line 513 uses uname and checks for the return value
    > equal to zero as success.  On Solaris, uname returns >= 0 on success.

diff --git a/hald/hald_runner.c b/hald/hald_runner.c
index f74dea1..400c90b 100644
--- a/hald/hald_runner.c
+++ b/hald/hald_runner.c
@@ -510,7 +510,7 @@ add_basic_env (DBusMessageIter * iter, const gchar * udi)
 	}
 #endif /* HAVE_CONKIT */
 
-	if (uname (&un) == 0) {
+	if (uname (&un) >= 0) {
 		char *sysname;
 
 		sysname = g_ascii_strdown (un.sysname, -1);


More information about the hal-commit mailing list