tags 274859 +patch thanks [martin f krafft] > I checked out the source and opening is not hard... but mdadm also > creates device nodes and uses S_ISBLK all over the place, so I don't > really know whether adding a "|| S_ISLNK" will fix it.
I didn't actually test this, but I honestly don't see why lstat() is used here, instead of stat() which is used everywhere else. Neil? Is there a good reason for lstat here? It apparently breaks on devfs. (Ref. http://bugs.debian.org/274859) Peter --- mdadm-1.9.0/mdopen.c~ 2005-02-03 18:45:23.000000000 -0600 +++ mdadm-1.9.0/mdopen.c 2005-05-23 19:34:12.000000000 -0500 @@ -97,7 +97,7 @@ return -1; } stb.st_mode = 0; - if (lstat(dev, &stb)==0 && ! S_ISBLK(stb.st_mode)) { + if (stat(dev, &stb)==0 && ! S_ISBLK(stb.st_mode)) { fprintf(stderr, Name ": %s is not a block device.\n", dev); return -1;
signature.asc
Description: Digital signature
