Fix return code check of pa_append_to_home_dir
authorFelipe Sateler <fsateler@debian.org>
Fri, 11 Sep 2015 00:42:48 +0000 (21:42 -0300)
committerTanu Kaskinen <tanuk@iki.fi>
Fri, 11 Sep 2015 11:22:15 +0000 (14:22 +0300)
It returns negative on error, zero on success.

Found by coverity

src/pulse/client-conf.c

index 83331f8..47fe183 100644 (file)
@@ -212,7 +212,7 @@ int pa_client_conf_load_cookie(pa_client_conf *c, uint8_t *cookie, size_t cookie
     if (r >= 0)
         return 0;
 
-    if (pa_append_to_home_dir(PA_NATIVE_COOKIE_FILE_FALLBACK, &fallback_path) > 0) {
+    if (pa_append_to_home_dir(PA_NATIVE_COOKIE_FILE_FALLBACK, &fallback_path) >= 0) {
         r = pa_authkey_load(fallback_path, false, cookie, cookie_length);
         pa_xfree(fallback_path);
         if (r >= 0)