Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]
Groups > linux.kernel > #1305025
| From | Arnd Bergmann <arnd@arndb.de> |
|---|---|
| Newsgroups | linux.kernel |
| Subject | [PATCH] ext4: fix ext4_ioctl_setproject return code |
| Date | 2016-01-08 23:00 +0100 |
| Message-ID | <qONeG-4j1-23@gated-at.bofh.it> (permalink) |
| Organization | linux.* mail to news gateway |
The newly introduced ext4_ioctl_setproject() functions comes in two
versions, one is fine and the other has an undefined return value,
as indicated by a compiler warning:
fs/ext4/ioctl.c:426:1: warning: control reaches end of non-void function [-Wreturn-type]
fs/ext4/ioctl.c: In function 'ext4_ioctl_setproject':
This changes the function to return zero on success, as the other
version does.
Signed-off-by: Arnd Bergmann <arnd@arndb.de>
Fixes: e14c6dbc6429 ("ext4: add FS_IOC_FSSETXATTR/FS_IOC_FSGETXATTR interface support")
---
Found on ARM randconfig testing on the last two linux-next releases.
diff --git a/fs/ext4/ioctl.c b/fs/ext4/ioctl.c
index c1d45aaa7d55..946560cb3fe6 100644
--- a/fs/ext4/ioctl.c
+++ b/fs/ext4/ioctl.c
@@ -423,6 +423,7 @@ static int ext4_ioctl_setproject(struct file *filp, __u32 projid)
{
if (projid != EXT4_DEF_PROJID)
return -EOPNOTSUPP;
+ return 0;
}
#endif
Back to linux.kernel | Previous | Next | Find similar | Unroll thread
[PATCH] ext4: fix ext4_ioctl_setproject return code Arnd Bergmann <arnd@arndb.de> - 2016-01-08 23:00 +0100
csiph-web