diff options
| author | Andi Kleen <ak@suse.de> | 2006-09-26 10:52:39 +0200 |
|---|---|---|
| committer | Andi Kleen <andi@basil.nowhere.org> | 2006-09-26 10:52:39 +0200 |
| commit | 95912008ba1fb9d0677c1ce5930aeb0e85ba5710 (patch) | |
| tree | e84c614f6aff984d105ce9f340f9474b613d567d /include/asm-x86_64/i387.h | |
| parent | 3022d734a54cbd2b65eea9a024564821101b4a9a (diff) | |
[PATCH] Add __must_check to copy_*_user
Following i386.
And also fix the two occurrences that caused warnings in arch/x86_64/*
Signed-off-by: Andi Kleen <ak@suse.de>
Diffstat (limited to 'include/asm-x86_64/i387.h')
| -rw-r--r-- | include/asm-x86_64/i387.h | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/include/asm-x86_64/i387.h b/include/asm-x86_64/i387.h index 60c0f4853fd..0217b74cc9f 100644 --- a/include/asm-x86_64/i387.h +++ b/include/asm-x86_64/i387.h @@ -137,8 +137,8 @@ static inline int save_i387_checking(struct i387_fxsave_struct __user *fx) #else : [fx] "cdaSDb" (fx), "0" (0)); #endif - if (unlikely(err)) - __clear_user(fx, sizeof(struct i387_fxsave_struct)); + if (unlikely(err) && __clear_user(fx, sizeof(struct i387_fxsave_struct))) + err = -EFAULT; /* No need to clear here because the caller clears USED_MATH */ return err; } |