Revert "elput: Missing pointer initialization."
authorChris Michael <cpmichael@osg.samsung.com>
Tue, 7 Jun 2016 16:09:20 +0000 (12:09 -0400)
committerChris Michael <cpmichael@osg.samsung.com>
Tue, 7 Jun 2016 16:09:20 +0000 (12:09 -0400)
Reverting this as the initialization of char *s was already set to
NULL in a previous patch, so the 'if (s)' check here is pointless
really as 'man 3 free' says passing NULL to free() is ok.

This reverts commit ad2e21b02399c02a748bb8b8ab0dc10f85e6662d.

src/lib/elput/elput_logind.c

index 799a26a..22cb1b0 100644 (file)
@@ -427,7 +427,7 @@ _logind_connect(Elput_Manager **manager, const char *seat, unsigned int tty)
    if (ret < 0)
      {
         ERR("Failed to get session seat");
-        if (s) free(s);
+        free(s);
         goto seat_err;
      }
    else if ((seat) && (s) && (strcmp(seat, s)))