Prevent buffer overflow in OF_getpropintarray().
authorkettenis <kettenis@openbsd.org>
Sat, 6 Aug 2022 08:59:28 +0000 (08:59 +0000)
committerkettenis <kettenis@openbsd.org>
Sat, 6 Aug 2022 08:59:28 +0000 (08:59 +0000)
ok jsg@

sys/dev/ofw/fdt.c

index 69e8065..def626a 100644 (file)
@@ -1,4 +1,4 @@
-/*     $OpenBSD: fdt.c,v 1.28 2022/01/09 05:42:45 jsg Exp $    */
+/*     $OpenBSD: fdt.c,v 1.29 2022/08/06 08:59:28 kettenis Exp $       */
 
 /*
  * Copyright (c) 2009 Dariusz Swiderski <sfires@sfires.net>
@@ -994,7 +994,7 @@ OF_getpropintarray(int handle, char *prop, uint32_t *buf, int buflen)
        if (len < 0 || (len % sizeof(uint32_t)))
                return -1;
 
-       for (i = 0; i < len / sizeof(uint32_t); i++)
+       for (i = 0; i < min(len, buflen) / sizeof(uint32_t); i++)
                buf[i] = betoh32(buf[i]);
 
        return len;