]> git.ipfire.org Git - thirdparty/systemd.git/commitdiff
tests: drop calls to unsetenv SYSTEMD_MEMPOOL 19116/head
authorZbigniew Jędrzejewski-Szmek <zbyszek@in.waw.pl>
Thu, 25 Mar 2021 11:19:18 +0000 (12:19 +0100)
committerZbigniew Jędrzejewski-Szmek <zbyszek@in.waw.pl>
Fri, 26 Mar 2021 14:54:58 +0000 (15:54 +0100)
Coverity was complaining that we don't check the return value, which we stopped
doing in 772e0a76f34914f6f81205e912e4744c6b23f704.

But it seems that we don't want those calls at all. The test was originally
added with the call in a6ee01caf3409ba9820e8824b9262fbac31a9f77, but I don't
see why we should override this. If the user wants to execute the test with
mempool disabled, we shouldn't ignore that.

Coverity CID#1444464, CID#1444466.

src/libsystemd/sd-device/test-sd-device-thread.c
src/libudev/test-udev-device-thread.c

index 763e98c114d837b1d356475e5afb6d4e0c336def..644f3c2aeeec87c8411892863fe0a774d07a7063 100644 (file)
@@ -30,8 +30,6 @@ int main(int argc, char *argv[]) {
         const char *key, *value;
         int r;
 
-        unsetenv("SYSTEMD_MEMPOOL");
-
         r = sd_device_new_from_syspath(&loopback, "/sys/class/net/lo");
         if (r < 0)
                 return handle_error_errno(r, "Failed to create loopback device object");
index 3f0341a698a41d690c41150f3c30555b1d5db3c9..c082fdca46843e03fc078df71b254e5854f3b43d 100644 (file)
@@ -28,8 +28,6 @@ int main(int argc, char *argv[]) {
         pthread_t t;
         int r;
 
-        unsetenv("SYSTEMD_MEMPOOL");
-
         loopback = udev_device_new_from_syspath(NULL, "/sys/class/net/lo");
         if (!loopback)
                 return handle_error_errno(errno, "Failed to create loopback device object");