From a02417645e4cf1563b01819ac92ab507277b5b56 Mon Sep 17 00:00:00 2001 From: rubidium Date: Thu, 31 May 2007 14:29:19 +0000 Subject: (svn r9998) -Fix (r9990): possible null pointer dereferences on MorphOS. --- src/fios.cpp | 7 ++++--- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a/src/fios.cpp b/src/fios.cpp index 3d475676e..cdde6e3cd 100644 --- a/src/fios.cpp +++ b/src/fios.cpp @@ -114,11 +114,12 @@ char *FiosBrowseTo(const FiosItem *item) case FIOS_TYPE_PARENT: /* Check for possible NULL ptr (not required for UNIXes, but AmigaOS-alikes) */ - if ((s = strrchr(path, PATHSEPCHAR)) != path) { + s = strrchr(path, PATHSEPCHAR); + if (s != NULL && s != path) { s[0] = '\0'; // Remove last path separator character, so we can go up one level. - s = strrchr(path, PATHSEPCHAR); - if (s != NULL) s[1] = '\0'; // go up a directory } + s = strrchr(path, PATHSEPCHAR); + if (s != NULL) s[1] = '\0'; // go up a directory #if defined(__MORPHOS__) || defined(__AMIGAOS__) /* On MorphOS or AmigaOS paths look like: "Volume:directory/subdirectory" */ else if ((s = strrchr(path, ':')) != NULL) s[1] = '\0'; -- cgit v1.2.3-54-g00ecf