]> git.ipfire.org Git - thirdparty/systemd.git/commitdiff
dissect-image: path[] is now defined for all metadata fields
authorLennart Poettering <lennart@poettering.net>
Mon, 4 Dec 2023 17:18:48 +0000 (18:18 +0100)
committerLennart Poettering <lennart@poettering.net>
Tue, 12 Dec 2023 15:19:05 +0000 (16:19 +0100)
There's no point in handling a case that never can happen.

src/shared/dissect-image.c

index 2687eafaf6b70fe9edc217f98458eb74a7ad3058..b63973cea8cfe9c4546ad7a9c955612a1414b106 100644 (file)
@@ -3390,10 +3390,7 @@ int dissected_image_acquire_metadata(DissectedImage *m, DissectImageFlags extra_
         assert(m);
 
         for (; n_meta_initialized < _META_MAX; n_meta_initialized ++) {
-                if (!paths[n_meta_initialized]) {
-                        fds[2*n_meta_initialized] = fds[2*n_meta_initialized+1] = -EBADF;
-                        continue;
-                }
+                assert(paths[n_meta_initialized]);
 
                 if (pipe2(fds + 2*n_meta_initialized, O_CLOEXEC) < 0) {
                         r = -errno;
@@ -3435,8 +3432,7 @@ int dissected_image_acquire_metadata(DissectedImage *m, DissectImageFlags extra_
                 for (unsigned k = 0; k < _META_MAX; k++) {
                         _cleanup_close_ int fd = -ENOENT;
 
-                        if (!paths[k])
-                                continue;
+                        assert(paths[k]);
 
                         fds[2*k] = safe_close(fds[2*k]);
 
@@ -3535,8 +3531,7 @@ int dissected_image_acquire_metadata(DissectedImage *m, DissectImageFlags extra_
         for (unsigned k = 0; k < _META_MAX; k++) {
                 _cleanup_fclose_ FILE *f = NULL;
 
-                if (!paths[k])
-                        continue;
+                assert(paths[k]);
 
                 fds[2*k+1] = safe_close(fds[2*k+1]);