diff options
author | Mike Marshall <hubcap@omnibond.com> | 2018-01-25 14:43:28 -0500 |
---|---|---|
committer | Mike Marshall <hubcap@omnibond.com> | 2018-02-06 16:38:12 -0500 |
commit | cf546ab6b189ab8b3acb26bbc2f0af4df21f16c7 (patch) | |
tree | c2adb4324042ad60aa1d52efcdcf01190f522cea /fs/orangefs/orangefs-utils.c | |
parent | 6bdfb48dae8e1b98735bf9387557298d17510be9 (diff) |
Orangefs: don't propogate whacky error codes
When we get an error return code from userspace (the client-core)
we check to make sure it is a valid code.
This patch maps the whacky return code to -EINVAL instead of
propagating garbage back up the call chain potentially resulting
in a hard-to-find train-wreck.
The client-core doesn't have any business returning whacky return
codes, but if it does, we don't want the kernel to crash as a result.
Signed-off-by: Mike Marshall <hubcap@omnibond.com>
Diffstat (limited to 'fs/orangefs/orangefs-utils.c')
-rw-r--r-- | fs/orangefs/orangefs-utils.c | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/fs/orangefs/orangefs-utils.c b/fs/orangefs/orangefs-utils.c index 3098c2ee7b5f..ccddfcab7fc0 100644 --- a/fs/orangefs/orangefs-utils.c +++ b/fs/orangefs/orangefs-utils.c @@ -537,6 +537,7 @@ int orangefs_normalize_to_errno(__s32 error_code) */ } else { gossip_err("orangefs: orangefs_normalize_to_errno: got error code which is not from ORANGEFS.\n"); + error_code = -EINVAL; } return error_code; } |