condition: add ConditionFileNotEmpty=

https://bugs.freedesktop.org/show_bug.cgi?id=54448

diff --git a/man/systemd.unit.xml b/man/systemd.unit.xml
index bf22ca9bd0fdfe584b3f768f88476002bf3c0637..00f60667175db2c63938cf05ab0fe6e31e70c249 100644 (file)
--- a/man/systemd.unit.xml
+++ b/man/systemd.unit.xml
@@ -755,6 +755,7 @@
                                 <term><varname>ConditionPathIsMountPoint=</varname></term>
                                 <term><varname>ConditionPathIsReadWrite=</varname></term>
                                 <term><varname>ConditionDirectoryNotEmpty=</varname></term>
+                                <term><varname>ConditionFileNotEmpty=</varname></term>
                                 <term><varname>ConditionFileIsExecutable=</varname></term>
                                 <term><varname>ConditionKernelCommandLine=</varname></term>
                                 <term><varname>ConditionVirtualization=</varname></term>
@@ -825,21 +826,28 @@
                                 (i.e. not mounted
                                 read-only).</para>
 
-                                <para><varname>ConditionFileIsExecutable=</varname>
+                                <para><varname>ConditionDirectoryNotEmpty=</varname>
                                 is similar to
                                 <varname>ConditionPathExists=</varname>
                                 but verifies whether a certain path
-                                exists, is a regular file and marked
-                                executable.</para>
+                                exists and is a non-empty
+                                directory.</para>
 
-                                <para><varname>ConditionDirectoryNotEmpty=</varname>
+                                <para><varname>ConditionFileNotEmpty=</varname>
                                 is similar to
                                 <varname>ConditionPathExists=</varname>
                                 but verifies whether a certain path
-                                exists and is a non-empty
-                                directory.</para>
+                                exists and refers to a regular file
+                                with a non-zero size.</para>
+
+                                <para><varname>ConditionFileIsExecutable=</varname>
+                                is similar to
+                                <varname>ConditionPathExists=</varname>
+                                but verifies whether a certain path
+                                exists, is a regular file and marked
+                                executable.</para>
 
-                                <para>Similarly,
+                                <para>Similar,
                                 <varname>ConditionKernelCommandLine=</varname>
                                 may be used to check whether a
                                 specific kernel command line option is

diff --git a/src/core/condition.c b/src/core/condition.c
index e5cda21c37d8e4f8bbae3cbbc7bd3d5178f40e07..32a37ccad61c514ff1807767535afad5462a134d 100644 (file)
--- a/src/core/condition.c
+++ b/src/core/condition.c
@@ -261,6 +261,15 @@ bool condition_test(Condition *c) {
                 return !(k == -ENOENT || k > 0) == !c->negate;
         }
 
+        case CONDITION_FILE_NOT_EMPTY: {
+                struct stat st;
+
+                if (stat(c->parameter, &st) < 0)
+                        return c->negate;
+
+                return (S_ISREG(st.st_mode) && st.st_size > 0) == !c->negate;
+        }
+
         case CONDITION_FILE_IS_EXECUTABLE: {
                 struct stat st;
 
@@ -350,6 +359,7 @@ static const char* const condition_type_table[_CONDITION_TYPE_MAX] = {
         [CONDITION_PATH_IS_MOUNT_POINT] = "ConditionPathIsMountPoint",
         [CONDITION_PATH_IS_READ_WRITE] = "ConditionPathIsReadWrite",
         [CONDITION_DIRECTORY_NOT_EMPTY] = "ConditionDirectoryNotEmpty",
+        [CONDITION_FILE_NOT_EMPTY] = "ConditionFileNotEmpty",
         [CONDITION_KERNEL_COMMAND_LINE] = "ConditionKernelCommandLine",
         [CONDITION_VIRTUALIZATION] = "ConditionVirtualization",
         [CONDITION_SECURITY] = "ConditionSecurity",

diff --git a/src/core/condition.h b/src/core/condition.h
index 55b331edd7d180562c9bbf99d6620a0a0b9777da..03954e40b3c00160d1917161ca408df1aacef4a9 100644 (file)
--- a/src/core/condition.h
+++ b/src/core/condition.h
@@ -33,6 +33,7 @@ typedef enum ConditionType {
         CONDITION_PATH_IS_MOUNT_POINT,
         CONDITION_PATH_IS_READ_WRITE,
         CONDITION_DIRECTORY_NOT_EMPTY,
+        CONDITION_FILE_NOT_EMPTY,
         CONDITION_FILE_IS_EXECUTABLE,
         CONDITION_KERNEL_COMMAND_LINE,
         CONDITION_VIRTUALIZATION,