summaryrefslogtreecommitdiffstats
path: root/fs/xfs
diff options
context:
space:
mode:
authorMarkus Elfring <elfring@users.sourceforge.net>2016-01-04 16:13:21 +1100
committerDave Chinner <david@fromorbit.com>2016-01-04 16:13:21 +1100
commita841b64df29b4c7e68ce564d752dfb2042db5404 (patch)
tree23345028be9659e59719c0744e85ff79b7124bd7 /fs/xfs
parentc5ab131ba0df8c1f1f52ffa6214d60aafeeddbd0 (diff)
downloadop-kernel-dev-a841b64df29b4c7e68ce564d752dfb2042db5404.zip
op-kernel-dev-a841b64df29b4c7e68ce564d752dfb2042db5404.tar.gz
XFS: Use a signed return type for suffix_kstrtoint()
The return type "unsigned long" was used by the suffix_kstrtoint() function even though it will eventually return a negative error code. Improve this implementation detail by using the type "int" instead. This issue was detected by using the Coccinelle software. Signed-off-by: Markus Elfring <elfring@users.sourceforge.net> Reviewed-by: Eric Sandeen <sandeen@redhat.com> Signed-off-by: Dave Chinner <david@fromorbit.com>
Diffstat (limited to 'fs/xfs')
-rw-r--r--fs/xfs/xfs_super.c2
1 files changed, 1 insertions, 1 deletions
diff --git a/fs/xfs/xfs_super.c b/fs/xfs/xfs_super.c
index 36bd882..b357757 100644
--- a/fs/xfs/xfs_super.c
+++ b/fs/xfs/xfs_super.c
@@ -137,7 +137,7 @@ static const match_table_t tokens = {
};
-STATIC unsigned long
+STATIC int
suffix_kstrtoint(char *s, unsigned int base, int *res)
{
int last, shift_left_factor = 0, _res;
OpenPOWER on IntegriCloud