diff options
author | Hisham Muhammad <hisham@gobolinux.org> | 2017-07-27 16:07:50 -0300 |
---|---|---|
committer | Hisham Muhammad <hisham@gobolinux.org> | 2017-07-27 16:07:50 -0300 |
commit | 09e241fb1271021e3615512debd3136891547562 (patch) | |
tree | 8245b6084607a34185cfa7d3d45f5228937020f1 /BatteryMeter.c | |
parent | 3975e9ce5cba0e3972b2ddab28c198e000441501 (diff) |
Security review: check results of snprintf.
Calls marked with xSnprintf shouldn't fail.
Abort program cleanly if any of them does.
Diffstat (limited to 'BatteryMeter.c')
-rw-r--r-- | BatteryMeter.c | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/BatteryMeter.c b/BatteryMeter.c index 0031050f..214248e2 100644 --- a/BatteryMeter.c +++ b/BatteryMeter.c @@ -40,7 +40,7 @@ static void BatteryMeter_updateValues(Meter * this, char *buffer, int len) { if (percent == -1) { this->values[0] = 0; - snprintf(buffer, len, "n/a"); + xSnprintf(buffer, len, "n/a"); return; } @@ -58,11 +58,11 @@ static void BatteryMeter_updateValues(Meter * this, char *buffer, int len) { } if (isOnAC == AC_PRESENT) { - snprintf(buffer, len, onAcText, percent); + xSnprintf(buffer, len, onAcText, percent); } else if (isOnAC == AC_ABSENT) { - snprintf(buffer, len, onBatteryText, percent); + xSnprintf(buffer, len, onBatteryText, percent); } else { - snprintf(buffer, len, unknownText, percent); + xSnprintf(buffer, len, unknownText, percent); } return; |