Thread (23 messages) 23 messages, 2 authors, 2012-02-22

Re: [PATCH]QNX6 filesystem (RO) driver

From: Al Viro <viro@ZenIV.linux.org.uk>
Date: 2012-02-17 18:35:50

On Fri, Feb 17, 2012 at 06:53:52PM +0100, Kai Bankett wrote:
quoted
Other than that (and assuming it survives your testing), I'm fine with
that variant.
At least no problems with all my test images. Just ran a complete test
(file reads, dir(s), filepermissions, symlink, hardlink, longfilenames,
longdirectories etc.) over (mmi/be/le/1k/2k blocksize images).

http://a6.ontika.net/patches/0001-fs-initial-qnx6fs-addition.patch.gz
Hmmm...

+                               if (len == de->de_size)
+                                       /* normal filename */
+                                       ino = qnx6_match(s, len, name, de);
+                                       if (ino)
+                                               goto found;
+                               else if ((de->de_size == 0xff) &&

Actually, gcc ought to have screamed at that.  Note that "else" here matches
the second "if", not the first one.  And you are not guaranteed that
ino has _ever_ been assigned anything; that comparison is deep in nasal
daemon country...
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help