diff options
author | Christian Göttsche <cgzones@googlemail.com> | 2020-11-24 18:31:03 +0100 |
---|---|---|
committer | Benny Baumann <BenBE@geshi.org> | 2020-12-06 16:03:44 +0100 |
commit | e1ce141bc3123c1dea7eb2ef908c8ebce945c72a (patch) | |
tree | 2da1f500777ba003c3ea1d73d97e259f0e099974 /linux/ZramMeter.c | |
parent | d9224c66a41e7918570b81d41495870c20380c1d (diff) |
Use size_t as len type for Meter_UpdateValues
Most of the time the parameter is passed to snprintf type functions
Diffstat (limited to 'linux/ZramMeter.c')
-rw-r--r-- | linux/ZramMeter.c | 14 |
1 files changed, 9 insertions, 5 deletions
diff --git a/linux/ZramMeter.c b/linux/ZramMeter.c index cee70586..35148871 100644 --- a/linux/ZramMeter.c +++ b/linux/ZramMeter.c @@ -11,7 +11,7 @@ static const int ZramMeter_attributes[] = { ZRAM }; -static void ZramMeter_updateValues(Meter* this, char* buffer, int size) { +static void ZramMeter_updateValues(Meter* this, char* buffer, size_t size) { int written; Platform_setZramValues(this); @@ -38,11 +38,15 @@ static void ZramMeter_updateValues(Meter* this, char* buffer, int size) { } *buffer++ = ')'; size--; - if ((size -= written) > 0) { - *buffer++ = '/'; - size--; - Meter_humanUnit(buffer, this->total, size); + if (size <= 0) { + return; + } + *buffer++ = '/'; + size--; + if (size <= 0) { + return; } + Meter_humanUnit(buffer, this->total, size); } static void ZramMeter_display(const Object* cast, RichString* out) { |