diff options
| author | Xavier Del Campo Romero <xavi.dcr@tutanota.com> | 2023-10-14 13:05:58 +0200 |
|---|---|---|
| committer | Xavier Del Campo Romero <xavi.dcr@tutanota.com> | 2023-10-14 13:08:25 +0200 |
| commit | d96e5685ee072d5fd80c562089c1080a53673187 (patch) | |
| tree | ddf3052004019d41728fd2f5b4558d69ec1bf2fb | |
| parent | 472b4ddbf151b44f5fc01c4c1752ce5529a0cf85 (diff) | |
| download | slcl-d96e5685ee072d5fd80c562089c1080a53673187.tar.gz | |
auth.c: Fix potential signed integer overflow
For platforms where int is a 16-bit data type, this operation might
overflow and possibly cause either unexpected behaviour and/or a
compiler warning.
Therefore, it is safer to promote each integer constant accordingly.
| -rw-r--r-- | auth.c | 4 |
1 files changed, 2 insertions, 2 deletions
@@ -422,7 +422,7 @@ int auth_quota(const struct auth *const a, const char *const user, *available = true; *quota = strtoull(qs, &end, 10); - const unsigned long long mul = 1024 * 1024; + const unsigned long long mul = 1024ul * 1024ul; if (errno || *end != '\0') { @@ -436,7 +436,7 @@ int auth_quota(const struct auth *const a, const char *const user, goto end; } - *quota *= 1024 * 1024; + *quota *= mul; break; } } |
