diff --git a/Applications/FileManager/DirectoryView.cpp b/Applications/FileManager/DirectoryView.cpp index d77c139f0d..0f5818c391 100644 --- a/Applications/FileManager/DirectoryView.cpp +++ b/Applications/FileManager/DirectoryView.cpp @@ -55,11 +55,20 @@ void DirectoryView::handle_activation(const GModelIndex& index) dbgprintf("on activation: %d,%d, this=%p, m_model=%p\n", index.row(), index.column(), this, m_model.ptr()); auto& node = model().node(index); auto path = node.full_path(model()); - if (node.is_directory()) { + + struct stat st; + if (stat(path.characters(), &st) < 0) { + perror("stat"); + return; + } + + if (S_ISDIR(st.st_mode)) { open(path); return; } - if (node.is_executable()) { + + // FIXME: This doesn't seem like the right way to fully detect executability. + if (st.st_mode & S_IXUSR) { if (fork() == 0) { int rc = execl(path.characters(), path.characters(), nullptr); if (rc < 0) diff --git a/Applications/FileManager/main.cpp b/Applications/FileManager/main.cpp index c9ce285c80..7634596c21 100644 --- a/Applications/FileManager/main.cpp +++ b/Applications/FileManager/main.cpp @@ -140,7 +140,7 @@ int main(int argc, char** argv) struct stat st; // If the directory no longer exists, we find a parent that does. - while (lstat(current_path.characters(), &st) != 0) { + while (stat(current_path.characters(), &st) != 0) { directory_view->open_parent_directory(); current_path = directory_view->path(); if (current_path == directories_model->root_path()) {