aboutsummaryrefslogtreecommitdiff
path: root/gnu/packages/patches/tofi-32bit-compat.patch
blob: 5f9089e589fdf4966f7de959b85ebfccc97585e8 (plain)
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
This patch is from upstream and shouldn't be needed in the next release.
https://github.com/philj56/tofi/commit/667075f0920da3c2b353fbce54b6430c195ef031.patch

From 667075f0920da3c2b353fbce54b6430c195ef031 Mon Sep 17 00:00:00 2001
From: Phil Jones <philj56@gmail.com>
Date: Sun, 30 Apr 2023 20:08:57 +0100
Subject: [PATCH] Replace `strto[u]l` with `strto[u]ll`.

On 32-bit systems, using `strtoul` was causing negative values for
unsigned options to be treated as valid, as the value was being parsed
as a 32-bit unsigned int, then cast to a 64-bit signed int, which
remained positive.
---
 src/color.c  | 8 ++++----
 src/config.c | 6 +++---
 2 files changed, 7 insertions(+), 7 deletions(-)

diff --git a/src/color.c b/src/color.c
index 4b6b356..b1d5e90 100644
--- a/src/color.c
+++ b/src/color.c
@@ -22,7 +22,7 @@ struct color hex_to_color(const char *hex)
 			hex[2], hex[2],
 			'\0'};
 		char *endptr;
-		tmp = strtol(str, &endptr, 16);
+		tmp = strtoll(str, &endptr, 16);
 		if (errno || *endptr != '\0' || tmp < 0) {
 			return (struct color) { -1, -1, -1, -1 };
 		}
@@ -37,14 +37,14 @@ struct color hex_to_color(const char *hex)
 			hex[3], hex[3],
 			'\0'};
 		char *endptr;
-		tmp = strtol(str, &endptr, 16);
+		tmp = strtoll(str, &endptr, 16);
 		if (errno || *endptr != '\0' || tmp < 0) {
 			return (struct color) { -1, -1, -1, -1 };
 		}
 		val = tmp;
 	} else if (len == 6) {
 		char *endptr;
-		tmp = strtol(hex, &endptr, 16);
+		tmp = strtoll(hex, &endptr, 16);
 		if (errno || *endptr != '\0' || tmp < 0) {
 			return (struct color) { -1, -1, -1, -1 };
 		}
@@ -53,7 +53,7 @@ struct color hex_to_color(const char *hex)
 		val |= 0xFFu;
 	} else if (len == 8) {
 		char *endptr;
-		tmp = strtol(hex, &endptr, 16);
+		tmp = strtoll(hex, &endptr, 16);
 		if (errno || *endptr != '\0' || tmp < 0) {
 			return (struct color) { -1, -1, -1, -1 };
 		}
diff --git a/src/config.c b/src/config.c
index 2b85028..556199d 100644
--- a/src/config.c
+++ b/src/config.c
@@ -1027,7 +1027,7 @@ uint32_t parse_uint32(const char *filename, size_t lineno, const char *str, bool
 {
 	errno = 0;
 	char *endptr;
-	int64_t ret = strtoul(str, &endptr, 0);
+	int64_t ret = strtoull(str, &endptr, 0);
 	if (endptr == str || *endptr != '\0') {
 		PARSE_ERROR(filename, lineno, "Failed to parse \"%s\" as unsigned int.\n", str);
 		if (err) {
@@ -1046,7 +1046,7 @@ int32_t parse_int32(const char *filename, size_t lineno, const char *str, bool *
 {
 	errno = 0;
 	char *endptr;
-	int64_t ret = strtol(str, &endptr, 0);
+	int64_t ret = strtoll(str, &endptr, 0);
 	if (endptr == str || *endptr != '\0') {
 		PARSE_ERROR(filename, lineno, "Failed to parse \"%s\" as int.\n", str);
 		if (err) {
@@ -1065,7 +1065,7 @@ struct uint32_percent parse_uint32_percent(const char *filename, size_t lineno,
 {
 	errno = 0;
 	char *endptr;
-	int64_t val = strtoul(str, &endptr, 0);
+	int64_t val = strtoull(str, &endptr, 0);
 	bool percent = false;
 	if (endptr == str || (*endptr != '\0' && *endptr != '%')) {
 		PARSE_ERROR(filename, lineno, "Failed to parse \"%s\" as unsigned int.\n", str);