| To: | xfs@xxxxxxxxxxx |
|---|---|
| Subject: | [PATCH] mkfs.xfs: fix fd validity check in get_topology |
| From: | Christoph Hellwig <hch@xxxxxxxxxxxxx> |
| Date: | Sat, 30 Jan 2010 14:52:26 -0500 |
| User-agent: | Mutt/1.5.19 (2009-01-05) |
Only negatie return values from open mean we failed to open the device.
Without this check we do not print the usage message when no device is
specified. This leads to a weird failure in xfstests 122.
Signed-off-by: Christoph Hellwig <hch@xxxxxx>
Index: xfsprogs-dev/mkfs/xfs_mkfs.c
===================================================================
--- xfsprogs-dev.orig/mkfs/xfs_mkfs.c 2010-01-30 20:44:03.505255109 +0100
+++ xfsprogs-dev/mkfs/xfs_mkfs.c 2010-01-30 20:44:20.579255807 +0100
@@ -455,7 +455,7 @@ static void get_topology(libxfs_init_t *
&ft->dsunit, &ft->dswidth, &ft->sectoralign);
fd = open(dfile, O_RDONLY);
/* If this fails we just fall back to BBSIZE */
- if (fd) {
+ if (fd >= 0) {
platform_findsizes(dfile, fd, &dummy, &bsz);
close(fd);
}
|
| <Prev in Thread] | Current Thread | [Next in Thread> |
|---|---|---|
| ||
| Previous by Date: | Re: [PATCH] xfstests 224: test aio hole-fill at 4g, Eric Sandeen |
|---|---|
| Next by Date: | Re: [PATCH] mkfs.xfs: fix fd validity check in get_topology, Eric Sandeen |
| Previous by Thread: | Do you need to purchase pc cases?, pccase1 |
| Next by Thread: | Re: [PATCH] mkfs.xfs: fix fd validity check in get_topology, Eric Sandeen |
| Indexes: | [Date] [Thread] [Top] [All Lists] |