summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
-rw-r--r--meta-oe/recipes-navigation/gpsd/gpsd/CVE-2025-67268.patch97
-rw-r--r--meta-oe/recipes-navigation/gpsd/gpsd_3.23.1.bb1
2 files changed, 98 insertions, 0 deletions
diff --git a/meta-oe/recipes-navigation/gpsd/gpsd/CVE-2025-67268.patch b/meta-oe/recipes-navigation/gpsd/gpsd/CVE-2025-67268.patch
new file mode 100644
index 0000000000..50dabf89d3
--- /dev/null
+++ b/meta-oe/recipes-navigation/gpsd/gpsd/CVE-2025-67268.patch
@@ -0,0 +1,97 @@
1From b3abe9d49d8fcc3f824d74a5c2cdcc30838f5904 Mon Sep 17 00:00:00 2001
2From: "Gary E. Miller" <gem@rellim.com>
3Date: Tue, 2 Dec 2025 19:36:04 -0800
4Subject: [PATCH] drivers/driver_nmea2000.c: Fix issue 356, skyview buffer
5 overrun.
6
7CVE: CVE-2025-67268
8Upstream-Status: Backport [https://github.com/ntpsec/gpsd/commit/dc966aa74c075d0a6535811d98628625cbfbe3f4]
9Signed-off-by: Gyorgy Sarvari <skandigraun@gmail.com>
10---
11 drivers/driver_nmea2000.c | 123 ++++++++++++++++++++++----------------
12 1 file changed, 71 insertions(+), 52 deletions(-)
13
14diff --git a/drivers/driver_nmea2000.c b/drivers/driver_nmea2000.c
15index 66959f0..70462b3 100644
16--- a/drivers/driver_nmea2000.c
17+++ b/drivers/driver_nmea2000.c
18@@ -89,14 +89,14 @@ static int scale_int(int32_t var, const int64_t factor)
19 static void print_data(struct gps_context_t *context,
20 unsigned char *buffer, int len, PGN *pgn)
21 {
22- if ((libgps_debuglevel >= LOG_IO) != 0) {
23- int l1, l2, ptr;
24+ if (LOG_IO <= libgps_debuglevel) {
25+ int l1;
26 char bu[128];
27
28- ptr = 0;
29- l2 = sprintf(&bu[ptr], "got data:%6u:%3d: ", pgn->pgn, len);
30+ int ptr = 0;
31+ int l2 = sprintf(&bu[ptr], "got data:%6u:%3d: ", pgn->pgn, len);
32 ptr += l2;
33- for (l1=0;l1<len;l1++) {
34+ for (l1 = 0; l1 < len; l1++) {
35 if (((l1 % 20) == 0) && (l1 != 0)) {
36 GPSD_LOG(LOG_IO, &context->errout, "%s\n", bu);
37 ptr = 0;
38@@ -434,6 +434,7 @@ static gps_mask_t hnd_129540(unsigned char *bu, int len, PGN *pgn,
39 struct gps_device_t *session)
40 {
41 int l1;
42+ int expected_len;
43
44 print_data(session->context, bu, len, pgn);
45 GPSD_LOG(LOG_DATA, &session->context->errout,
46@@ -441,24 +442,39 @@ static gps_mask_t hnd_129540(unsigned char *bu, int len, PGN *pgn,
47
48 session->driver.nmea2000.sid[2] = bu[0];
49 session->gpsdata.satellites_visible = (int)bu[2];
50+ if (MAXCHANNELS <= session->gpsdata.satellites_visible) {
51+ // Handle a CVE for overrunning skyview[]
52+ GPSD_LOG(LOG_WARN, &session->context->errout,
53+ "pgn %6d(%3d): Too many sats %d\n",
54+ pgn->pgn, session->driver.nmea2000.unit,
55+ session->gpsdata.satellites_visible);
56+ session->gpsdata.satellites_visible = MAXCHANNELS;
57+ }
58+ expected_len = 3 + (12 * session->gpsdata.satellites_visible);
59+ if (len != expected_len) {
60+ GPSD_LOG(LOG_WARN, &session->context->errout,
61+ "pgn %6d(%3d): wrong length %d s/b %d\n",
62+ pgn->pgn, session->driver.nmea2000.unit,
63+ len, expected_len);
64+ return 0;
65+ }
66
67 memset(session->gpsdata.skyview, '\0', sizeof(session->gpsdata.skyview));
68- for (l1=0;l1<session->gpsdata.satellites_visible;l1++) {
69- int svt;
70- double azi, elev, snr;
71-
72- elev = getles16(bu, 3+12*l1+1) * 1e-4 * RAD_2_DEG;
73- azi = getleu16(bu, 3+12*l1+3) * 1e-4 * RAD_2_DEG;
74- snr = getles16(bu, 3+12*l1+5) * 1e-2;
75+ for (l1 = 0; l1 < session->gpsdata.satellites_visible; l1++) {
76+ int offset = 3 + (12 * l1);
77+ double elev = getles16(bu, offset + 1) * 1e-4 * RAD_2_DEG;
78+ double azi = getleu16(bu, offset + 3) * 1e-4 * RAD_2_DEG;
79+ double snr = getles16(bu, offset + 5) * 1e-2;
80
81- svt = (int)(bu[3+12*l1+11] & 0x0f);
82+ int svt = (int)(bu[offset + 11] & 0x0f);
83
84- session->gpsdata.skyview[l1].elevation = (short) (round(elev));
85- session->gpsdata.skyview[l1].azimuth = (short) (round(azi));
86+ session->gpsdata.skyview[l1].elevation = elev;
87+ session->gpsdata.skyview[l1].azimuth = azi;
88 session->gpsdata.skyview[l1].ss = snr;
89- session->gpsdata.skyview[l1].PRN = (short)bu[3+12*l1+0];
90+ session->gpsdata.skyview[l1].PRN = (int16_t)bu[offset];
91 session->gpsdata.skyview[l1].used = false;
92- if ((svt == 2) || (svt == 5)) {
93+ if ((2 == svt) ||
94+ (5 == svt)) {
95 session->gpsdata.skyview[l1].used = true;
96 }
97 }
diff --git a/meta-oe/recipes-navigation/gpsd/gpsd_3.23.1.bb b/meta-oe/recipes-navigation/gpsd/gpsd_3.23.1.bb
index 410db92bd0..87c70d3683 100644
--- a/meta-oe/recipes-navigation/gpsd/gpsd_3.23.1.bb
+++ b/meta-oe/recipes-navigation/gpsd/gpsd_3.23.1.bb
@@ -7,6 +7,7 @@ PROVIDES = "virtual/gpsd"
7 7
8SRC_URI = "${SAVANNAH_GNU_MIRROR}/${BPN}/${BP}.tar.gz \ 8SRC_URI = "${SAVANNAH_GNU_MIRROR}/${BPN}/${BP}.tar.gz \
9 file://gpsd.init \ 9 file://gpsd.init \
10 file://CVE-2025-67268.patch \
10 " 11 "
11SRC_URI[sha256sum] = "0b991ce9a46538c4ea450f7a8ee428ff44fb4f8d665fddf2ffe40fe0ae9a6c09" 12SRC_URI[sha256sum] = "0b991ce9a46538c4ea450f7a8ee428ff44fb4f8d665fddf2ffe40fe0ae9a6c09"
12 13