add Mirek's fetchmail-lmtp.patch to fix an operator precedence bug '||' vs ','

[~andy/fetchmail] / fetchmail.c

diff --git a/fetchmail.c b/fetchmail.c
index f150e8da6ffe82fe596260adac6a8cd3d7545815..868a45c183ff9a3a92a5e38cb73c265ac014acba 100644 (file)
--- a/fetchmail.c
+++ b/fetchmail.c
@@ -1254,9 +1254,9 @@ static int load_params(int argc, char **argv, int optind)
                {
                    char        *cp;
 
-                   if (!(cp = strrchr((char *)idp->id, '/')) ||
-                           ++cp, (0 == strcmp(cp, SMTP_PORT))
-                               || servport(cp) == SMTP_PORT_NUM)
+                   if (!(cp = strrchr((char *)idp->id, '/'))
+                       || (0 == strcmp(cp + 1, SMTP_PORT))
+                       || servport(cp + 1) == SMTP_PORT_NUM)
                    {
                        (void) fprintf(stderr,
                                       GT_("%s configuration invalid, LMTP can't use default SMTP port\n"),