diff options
| author |   Patrick Palka <patrick@parcs.ath.cx> | 2013-10-11 13:11:55 -0400 |
|---|---|---|
| committer |   John Stultz <john.stultz@linaro.org> | 2013-10-18 16:45:58 -0700 |
| commit | 891292a767c2453af0e5be9465e95b06b4b29ebe (patch) | |
| tree | 86f9d3856dd7cb253f2b635274b78e81da7e8fe5 /kernel/time/tick-internal.h | |
| parent | timekeeping: Fix some trivial typos in comments (diff) | |
| download | linux-dev-891292a767c2453af0e5be9465e95b06b4b29ebe.tar.xz linux-dev-891292a767c2453af0e5be9465e95b06b4b29ebe.zip | |
time: Fix signedness bug in sysfs_get_uname() and its callers
sysfs_get_uname() is erroneously declared as returning size_t even
though it may return a negative value, specifically -EINVAL. Its
callers then check whether its return value is less than zero and indeed
that is never the case for size_t.
This patch changes sysfs_get_uname() to return ssize_t and makes sure
its callers use ssize_t accordingly.
Signed-off-by: Patrick Palka <patrick@parcs.ath.cx>
[jstultz: Didn't apply cleanly, as a similar partial fix was also applied
so had to resolve the collisions]
Signed-off-by: John Stultz <john.stultz@linaro.org>
Diffstat (limited to 'kernel/time/tick-internal.h')
| -rw-r--r-- | kernel/time/tick-internal.h | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/kernel/time/tick-internal.h b/kernel/time/tick-internal.h index bc906cad709b..18e71f7fbc2a 100644 --- a/kernel/time/tick-internal.h +++ b/kernel/time/tick-internal.h @@ -31,7 +31,7 @@ extern void tick_install_replacement(struct clock_event_device *dev); extern void clockevents_shutdown(struct clock_event_device *dev); -extern size_t sysfs_get_uname(const char *buf, char *dst, size_t cnt); +extern ssize_t sysfs_get_uname(const char *buf, char *dst, size_t cnt); /* * NO_HZ / high resolution timer shared code |