]> git.ipfire.org Git - thirdparty/systemd.git/commitdiff
core: sd_bus_error() already checks for NULL, no need to duplicate check
authorLennart Poettering <lennart@poettering.net>
Tue, 19 May 2015 15:56:44 +0000 (17:56 +0200)
committerLennart Poettering <lennart@poettering.net>
Tue, 19 May 2015 15:56:44 +0000 (17:56 +0200)
src/core/transaction.c

index 223f691534a5fdba4404f58cb93022ed4fc098dc..981705718526bb0b6b8984167a104e73dcb16338 100644 (file)
@@ -921,9 +921,7 @@ int transaction_add_job_and_dependencies(
                                 r = transaction_add_job_and_dependencies(tr, type, dep, ret, false, override, false, false, ignore_order, e);
                                 if (r < 0) {
                                         log_unit_warning(dep, "Cannot add dependency job for, ignoring: %s", bus_error_message(e, r));
-
-                                        if (e)
-                                                sd_bus_error_free(e);
+                                        sd_bus_error_free(e);
                                 }
                         }
 
@@ -938,8 +936,7 @@ int transaction_add_job_and_dependencies(
                                         if (r != -EBADR)
                                                 goto fail;
 
-                                        if (e)
-                                                sd_bus_error_free(e);
+                                        sd_bus_error_free(e);
                                 }
                         }
 
@@ -949,8 +946,7 @@ int transaction_add_job_and_dependencies(
                                         if (r != -EBADR)
                                                 goto fail;
 
-                                        if (e)
-                                                sd_bus_error_free(e);
+                                        sd_bus_error_free(e);
                                 }
                         }
 
@@ -961,9 +957,7 @@ int transaction_add_job_and_dependencies(
                                                       r == -EADDRNOTAVAIL ? LOG_DEBUG : LOG_WARNING, r,
                                                       "Cannot add dependency job, ignoring: %s",
                                                       bus_error_message(e, r));
-
-                                        if (e)
-                                                sd_bus_error_free(e);
+                                        sd_bus_error_free(e);
                                 }
                         }
 
@@ -974,9 +968,7 @@ int transaction_add_job_and_dependencies(
                                                       r == -EADDRNOTAVAIL ? LOG_DEBUG : LOG_WARNING, r,
                                                       "Cannot add dependency job, ignoring: %s",
                                                       bus_error_message(e, r));
-
-                                        if (e)
-                                                sd_bus_error_free(e);
+                                        sd_bus_error_free(e);
                                 }
                         }
 
@@ -986,8 +978,7 @@ int transaction_add_job_and_dependencies(
                                         if (r != -EBADR)
                                                 goto fail;
 
-                                        if (e)
-                                                sd_bus_error_free(e);
+                                        sd_bus_error_free(e);
                                 }
                         }
 
@@ -998,9 +989,7 @@ int transaction_add_job_and_dependencies(
                                                       r == -EADDRNOTAVAIL ? LOG_DEBUG : LOG_WARNING, r,
                                                       "Cannot add dependency job, ignoring: %s",
                                                       bus_error_message(e, r));
-
-                                        if (e)
-                                                sd_bus_error_free(e);
+                                        sd_bus_error_free(e);
                                 }
                         }
 
@@ -1010,8 +999,7 @@ int transaction_add_job_and_dependencies(
                                         if (r != -EBADR)
                                                 goto fail;
 
-                                        if (e)
-                                                sd_bus_error_free(e);
+                                        sd_bus_error_free(e);
                                 }
                         }
 
@@ -1021,9 +1009,7 @@ int transaction_add_job_and_dependencies(
                                         log_unit_warning(dep,
                                                          "Cannot add dependency job, ignoring: %s",
                                                          bus_error_message(e, r));
-
-                                        if (e)
-                                                sd_bus_error_free(e);
+                                        sd_bus_error_free(e);
                                 }
                         }
 
@@ -1060,9 +1046,7 @@ int transaction_add_job_and_dependencies(
                                         log_unit_warning(dep,
                                                          "Cannot add dependency reload job, ignoring: %s",
                                                          bus_error_message(e, r));
-
-                                        if (e)
-                                                sd_bus_error_free(e);
+                                        sd_bus_error_free(e);
                                 }
                         }
                 }