git: 2c52512caf6e - main - pnglite: should use ntohl
Toomas Soome
tsoome at FreeBSD.org
Mon Jan 11 22:58:01 UTC 2021
The branch main has been updated by tsoome:
URL: https://cgit.FreeBSD.org/src/commit/?id=2c52512caf6ec10f49038e6884b9c2aea905cc4e
commit 2c52512caf6ec10f49038e6884b9c2aea905cc4e
Author: Toomas Soome <tsoome at FreeBSD.org>
AuthorDate: 2021-01-11 19:42:44 +0000
Commit: Toomas Soome <tsoome at FreeBSD.org>
CommitDate: 2021-01-11 22:56:35 +0000
pnglite: should use ntohl
Replace manual conversion with ntohl()
---
contrib/pnglite/pnglite.c | 15 ++++-----------
1 file changed, 4 insertions(+), 11 deletions(-)
diff --git a/contrib/pnglite/pnglite.c b/contrib/pnglite/pnglite.c
index e3cc2f866c6f..f517b6bdcaa5 100644
--- a/contrib/pnglite/pnglite.c
+++ b/contrib/pnglite/pnglite.c
@@ -52,12 +52,12 @@ file_read(png_t *png, void *out, size_t size, size_t numel)
static int
file_read_ul(png_t *png, unsigned *out)
{
- uint8_t buf[4];
+ uint32_t buf;
- if (file_read(png, buf, 1, 4) != 4)
+ if (file_read(png, &buf, 1, 4) != 4)
return (PNG_FILE_ERROR);
- *out = (buf[0]<<24) | (buf[1]<<16) | (buf[2]<<8) | buf[3];
+ *out = ntohl(buf);
return (PNG_NO_ERROR);
}
@@ -65,14 +65,7 @@ file_read_ul(png_t *png, unsigned *out)
static unsigned
get_ul(uint8_t *buf)
{
- unsigned result;
- uint8_t foo[4];
-
- memcpy(foo, buf, 4);
-
- result = (foo[0]<<24) | (foo[1]<<16) | (foo[2]<<8) | foo[3];
-
- return (result);
+ return (ntohl(*(uint32_t *)buf));
}
static int
More information about the dev-commits-src-all
mailing list