From daba307035faf4cc01f1ccb883b21281d91234cc Mon Sep 17 00:00:00 2001 From: Ralf Baechle Date: Fri, 28 Jul 2017 14:10:07 +0200 Subject: treewide: Kill assignments in if conditions. Somewhat hard to read and the code base already has many overlong lines. Found with below spatch file and some manual editing in ax25/access.c to restore a comment lost by spatch. @parens@ expression E, F, G; binary operator X; statement S; @@ - if ((E = F) X G) + E = F; + if (E X G) S Signed-off-by: Ralf Baechle --- tcpip/rip98d.c | 18 ++++++++++++------ tcpip/rip98r.c | 6 ++++-- tcpip/ttylinkd.c | 27 ++++++++++++++++++--------- 3 files changed, 34 insertions(+), 17 deletions(-) (limited to 'tcpip') diff --git a/tcpip/rip98d.c b/tcpip/rip98d.c index 4b7be6b..c0543d0 100644 --- a/tcpip/rip98d.c +++ b/tcpip/rip98d.c @@ -180,7 +180,8 @@ static int read_routes(void) first_route = NULL; } - if ((fp = fopen(PROC_IP_ROUTE_FILE, "r")) == NULL) { + fp = fopen(PROC_IP_ROUTE_FILE, "r"); + if (fp == NULL) { if (logging) syslog(LOG_ERR, "error cannot open %s\n", PROC_IP_ROUTE_FILE); return FALSE; @@ -213,7 +214,8 @@ static int read_routes(void) } } - if ((route = malloc(sizeof(struct route_struct))) == NULL) { + route = malloc(sizeof(struct route_struct)); + if (route == NULL) { if (logging) syslog(LOG_ERR, "out of memory !\n"); return FALSE; @@ -239,15 +241,18 @@ static int load_dests(void) char buffer[255], *s; FILE *fp; - if ((fp = fopen(CONF_RIP98D_FILE, "r")) == NULL) { + fp = fopen(CONF_RIP98D_FILE, "r"); + if (fp == NULL) { fprintf(stderr, "rip98d: cannot open config file\n"); return FALSE; } while (fgets(buffer, 255, fp) != NULL) { - if ((s = strchr(buffer, '\n')) != NULL) *s = '\0'; + s = strchr(buffer, '\n'); + if (s != NULL) *s = '\0'; - if ((host = gethostbyname(buffer)) == NULL) { + host = gethostbyname(buffer); + if (host == NULL) { fprintf(stderr, "rip98d: cannot resolve name %s\n", buffer); fclose(fp); return FALSE; @@ -311,7 +316,8 @@ int main(int argc, char **argv) signal(SIGTERM, terminate); - if ((s = socket(AF_INET, SOCK_DGRAM, 0)) < 0) { + s = socket(AF_INET, SOCK_DGRAM, 0); + if (s < 0) { perror("rip98d: socket"); return 1; } diff --git a/tcpip/rip98r.c b/tcpip/rip98r.c index b1f283c..57afe4e 100644 --- a/tcpip/rip98r.c +++ b/tcpip/rip98r.c @@ -160,7 +160,8 @@ void receive_routes(int s) case ADDITIONAL_ROUTE: if (!found) { - if ((new = malloc(sizeof(struct route_struct))) == NULL) { + new = malloc(sizeof(struct route_struct)); + if (new == NULL) { if (logging) syslog(LOG_ERR, "out of memory\n"); return; @@ -186,7 +187,8 @@ void receive_routes(int s) } if (!matched) { - if ((new = malloc(sizeof(struct route_struct))) == NULL) { + new = malloc(sizeof(struct route_struct)); + if (new == NULL) { if (logging) syslog(LOG_ERR, "out of memory\n"); return; diff --git a/tcpip/ttylinkd.c b/tcpip/ttylinkd.c index c11ffa6..200cfdd 100644 --- a/tcpip/ttylinkd.c +++ b/tcpip/ttylinkd.c @@ -238,7 +238,8 @@ int main(int argc, char *argv[]) /* Get the remote address, or use ours if remote isn't specified */ if (sysop_host) { - if ((rhe = gethostbyname(sysop_host)) == NULL) + rhe = gethostbyname(sysop_host); + if (rhe == NULL) { syslog(LOG_DAEMON | LOG_CRIT, "main(): gethostbyname failed."); exit(1); @@ -246,7 +247,8 @@ int main(int argc, char *argv[]) } else { - if ((rhe = gethostbyname(hostname)) == NULL) + rhe = gethostbyname(hostname); + if (rhe == NULL) { syslog(LOG_DAEMON | LOG_CRIT, "main(): gethostbyname failed."); exit(1); @@ -255,7 +257,8 @@ int main(int argc, char *argv[]) memcpy((char*)&rem_addr, rhe->h_addr, rhe->h_length); /* Get our local address */ - if ((phe = gethostbyname(hostname)) == NULL) + phe = gethostbyname(hostname); + if (phe == NULL) { syslog(LOG_DAEMON | LOG_CRIT, "main(): gethostbyname failed."); exit(1); @@ -270,7 +273,8 @@ int main(int argc, char *argv[]) msg_sin->sin_port = htons(0); memcpy((char*)&(msg_sin->sin_addr), phe->h_addr, phe->h_length); - if ((skt = socket(PF_INET, SOCK_STREAM, 0)) < 0) + skt = socket(PF_INET, SOCK_STREAM, 0); + if (skt < 0) { syslog(LOG_DAEMON | LOG_CRIT, "main(): socket() failed."); exit(1); @@ -296,7 +300,8 @@ int main(int argc, char *argv[]) memcpy((char*)&ctl_sin.sin_addr, phe->h_addr, phe->h_length); ctl_sin.sin_port = htons(0); - if ((ctl_skt = socket(PF_INET, SOCK_DGRAM, 0)) < 0) + ctl_skt = socket(PF_INET, SOCK_DGRAM, 0); + if (ctl_skt < 0) { syslog(LOG_DAEMON | LOG_CRIT, "main(): socket() while attempting to create control socket."); close(skt); @@ -383,7 +388,8 @@ int main(int argc, char *argv[]) signal(SIGALRM, alarm_handle); alarm(30); - if ((new_skt = accept(skt, NULL, NULL)) < 0) + new_skt = accept(skt, NULL, NULL); + if (new_skt < 0) { if (errno == EINTR) { @@ -659,7 +665,8 @@ static void do_talk(int skt) if (FD_ISSET(skt, &fdvar)) { - if ((i = read(skt, inbuf, 256)) <= 0) + i = read(skt, inbuf, 256); + if (i <= 0) { strcpy(inbuf,"Sysop closed connection.\n"); write(STDOUT_FILENO, inbuf, strlen(inbuf)); @@ -674,7 +681,8 @@ static void do_talk(int skt) } if (FD_ISSET(STDIN_FILENO, &fdvar)) { - if ((i = read(STDIN_FILENO, outbuf, 256)) <= 0) + i = read(STDIN_FILENO, outbuf, 256); + if (i <= 0) { strcpy(outbuf, "User closed connection.\n"); (void) write(skt, outbuf, strlen(outbuf)); @@ -704,7 +712,8 @@ static void read_config_file(int dummy) char param[20], value[108]; FILE *fp; - if ( (fp = fopen(config_file, "r")) == NULL) { + fp = fopen(config_file, "r"); + if (fp == NULL) { syslog(LOG_DAEMON | LOG_ERR, "Cannot find configuration file: %s (%m)\n",config_file); return; } -- cgit v1.2.3