diff options
author | Max Horn | 2006-04-27 17:21:33 +0000 |
---|---|---|
committer | Max Horn | 2006-04-27 17:21:33 +0000 |
commit | cf13cf3a706dba93b39b3fdc1176b91c569575d1 (patch) | |
tree | 62d1a920476a071928f11f456f8d0a69b523fbc4 | |
parent | 5c2dd8da2412cad1ca75dbdd46702d7fbf1ccad5 (diff) | |
download | scummvm-rg350-cf13cf3a706dba93b39b3fdc1176b91c569575d1.tar.gz scummvm-rg350-cf13cf3a706dba93b39b3fdc1176b91c569575d1.tar.bz2 scummvm-rg350-cf13cf3a706dba93b39b3fdc1176b91c569575d1.zip |
I still don't know whether this is the correct fix for WindowsFilesystemNode, but at least with it, one definite and serious bug in the code (using an uninitialised variable) has been fixed, the end result hardly will be worse
svn-id: r22192
-rw-r--r-- | backends/fs/windows/windows-fs.cpp | 4 |
1 files changed, 4 insertions, 0 deletions
diff --git a/backends/fs/windows/windows-fs.cpp b/backends/fs/windows/windows-fs.cpp index 06dbb57c05..837fed1e72 100644 --- a/backends/fs/windows/windows-fs.cpp +++ b/backends/fs/windows/windows-fs.cpp @@ -158,6 +158,10 @@ WindowsFilesystemNode::WindowsFilesystemNode(const String &p) { _isValid = true; _isDirectory = ((fileAttribs & FILE_ATTRIBUTE_DIRECTORY) != 0); } + // FIXME: Is the following correct? Should we maybe check for path being + // empty and set it to true in this case? + // In any case, not setting _isPseudoRoot at all certainly is a bug. + _isPseudoRoot = false; } FSList WindowsFilesystemNode::listDir(ListMode mode) const { |