Revert "coredumpctl: Don't treat no coredumps as failure"

This reverts commit dfe79b9ed21b0feeb5a120e8b994f46cff7cf5b0.

diff --git a/man/coredumpctl.xml b/man/coredumpctl.xml
index 6db9bca0681a24d1298fc8870e4e2cce09fc415e..6179d6b760d95e69d6ed4e95641dcdd07d00227e 100644 (file)
--- a/man/coredumpctl.xml
+++ b/man/coredumpctl.xml
@@ -389,7 +389,10 @@
 
   <refsect1>
     <title>Exit status</title>
-    <para>On success, 0 is returned; otherwise, a non-zero failure code is returned.</para>
+    <para>On success, 0 is returned; otherwise, a non-zero failure
+    code is returned. Not finding any matching core dumps is treated as
+    failure.
+    </para>
   </refsect1>
 
   <refsect1>

diff --git a/src/coredump/coredumpctl.c b/src/coredump/coredumpctl.c
index 4a25a2fb703a1f78316bd677e66fcdd4adb9e41e..e7d0dd34c0d104d88f8db615c1ce41af41dd750a 100644 (file)
--- a/src/coredump/coredumpctl.c
+++ b/src/coredump/coredumpctl.c
@@ -963,10 +963,9 @@ static int dump_list(int argc, char **argv, void *userdata) {
                 }
 
                 if (!arg_field && n_found <= 0) {
-                        if (!arg_quiet && !sd_json_format_enabled(arg_json_format_flags))
+                        if (!arg_quiet)
                                 log_notice("No coredumps found.");
-
-                        return 0;
+                        return -ESRCH;
                 }
         }