kolyshkin commented on this pull request.


> +     struct dirent *entry;
+       while ((entry = readdir(dir)) != NULL) {
+               fd = atoi(entry->d_name);
+               if (fd >= min_fd)
+                       set_cloexec(fd);
+       }
+       closedir(dir);
+
+       return;
+
+fallback:
+       // iterate over all possible fds
+       for (fd = min_fd; fd < open_max; fd++)
+               set_cloexec(fd);
+}
+
 static int Pexec(lua_State *L)                 /** exec(path,[args]) */

Yes, ideally those two places need to be unified

-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/rpm-software-management/rpm/pull/444#discussion_r189105974
_______________________________________________
Rpm-maint mailing list
Rpm-maint@lists.rpm.org
http://lists.rpm.org/mailman/listinfo/rpm-maint

Reply via email to