From 897886a68ce9146ddf990e675a0dee76b294a6e7 Mon Sep 17 00:00:00 2001 From: Jeremy Sowden Date: Fri, 28 Jun 2019 16:11:21 +0100 Subject: [PATCH] wmbiff: fixed endianness problems parsing server-ports. `regulo_atoi` expects a pointer-to-int and `PCU.serverPort` is a `uint16_t`, so `&PCU.serverPort` is not compatible and we need to use an `int` temporary variable to avoid endianness problems. Signed-off-by: Jeremy Sowden --- wmbiff/wmbiff/Imap4Client.c | 12 ++++++++++-- 1 file changed, 10 insertions(+), 2 deletions(-) diff --git a/wmbiff/wmbiff/Imap4Client.c b/wmbiff/wmbiff/Imap4Client.c index ea24dd9..395db2b 100644 --- a/wmbiff/wmbiff/Imap4Client.c +++ b/wmbiff/wmbiff/Imap4Client.c @@ -501,13 +501,19 @@ int imap4Create( /*@notnull@ */ Pop3 pc, const char *const str) NULL }; char *unaliased_str; + /* + * regulo_atoi expects a pointer-to-int and pop_imap.serverPort is a + * uint16_t, so &pop_imap.serverPort is not compatible and we need to use an + * int temporary variable to avoid endianness problems. + */ + int serverPort; struct regulo regulos[] = { {1, PCU.userName, regulo_strcpy}, {2, PCU.password, regulo_strcpy}, {3, PCU.serverName, regulo_strcpy}, {4, pc->path, regulo_strcpy_skip1}, - {7, &PCU.serverPort, regulo_atoi}, + {7, &serverPort, regulo_atoi}, {9, PCU.authList, regulo_strcpy_tolower}, {0, NULL, NULL} }; @@ -536,7 +542,7 @@ int imap4Create( /*@notnull@ */ Pop3 pc, const char *const str) } /* defaults */ - PCU.serverPort = (PCU.dossl != 0) ? 993 : 143; + serverPort = (PCU.dossl != 0) ? 993 : 143; PCU.authList[0] = '\0'; /* argh, str and pc->path are aliases, so we can't just write the default @@ -559,6 +565,8 @@ int imap4Create( /*@notnull@ */ Pop3 pc, const char *const str) return -1; } + PCU.serverPort = serverPort; + PCU.password_len = strlen(PCU.password); if (PCU.password[0] == '\0') { PCU.interactive_password = 1; -- 2.11.4.GIT