misc cleanup

removing the branches checking for "." has no effect,
as calling openat on this just reopens the previous path element.

1 files changed, 1 insertions, 6 deletions

diff --git a/src/firejail/util.c b/src/firejail/util.c
index 0e869ef7a..dd298a31a 100644
--- a/src/firejail/util.c
+++ b/src/firejail/util.c
@@ -1163,11 +1163,6 @@ int safe_fd(const char *path, int flags) {
         char *tok = strtok(dup, "/");
         assert(tok);
         while (tok) {
-                // skip all "/./"
-                if (strcmp(tok, ".") == 0) {
-                        tok = strtok(NULL, "/");
-                        continue;
-                }
                 // open the element, assuming it is a directory; this fails with ENOTDIR if it is a symbolic link
                 fd = openat(parentfd, tok, O_PATH|O_DIRECTORY|O_NOFOLLOW|O_CLOEXEC);
                 if (fd == -1) {
@@ -1267,7 +1262,7 @@ int invalid_sandbox(const pid_t pid) {
 }
 
 int has_handler(pid_t pid, int signal) {
-        if (signal > 0) {
+        if (signal > 0 && signal <= SIGRTMAX) {
                 char *fname;
                 if (asprintf(&fname, "/proc/%d/status", pid) == -1)
                         errExit("asprintf");