diff options
| author | David M. Syzdek <david.syzdek@acsalaska.com> | 2022-07-21 23:32:29 -0800 |
|---|---|---|
| committer | David M. Syzdek <david.syzdek@acsalaska.com> | 2022-07-21 23:32:29 -0800 |
| commit | 1610e3b15c634f5f70ad75a7711cb0b899c0e7a9 (patch) | |
| tree | 3a9426042a6820ca40a28759032627def75b0951 | |
| parent | 63c92e872110e37a6319566446bf02984210c79c (diff) | |
| download | akcom-udpecho-1610e3b15c634f5f70ad75a7711cb0b899c0e7a9.tar.gz akcom-udpecho-1610e3b15c634f5f70ad75a7711cb0b899c0e7a9.tar.bz2 akcom-udpecho-1610e3b15c634f5f70ad75a7711cb0b899c0e7a9.tar.xz akcom-udpecho-1610e3b15c634f5f70ad75a7711cb0b899c0e7a9.zip | |
akcom-udpechod: resolving xcode analyzer warnings
| -rw-r--r-- | src/akcom-udpechod.c | 22 |
1 files changed, 11 insertions, 11 deletions
diff --git a/src/akcom-udpechod.c b/src/akcom-udpechod.c index 6c40876..ea183bc 100644 --- a/src/akcom-udpechod.c +++ b/src/akcom-udpechod.c | |||
| @@ -502,7 +502,7 @@ my_daemonize( | |||
| 502 | // check for existing instance | 502 | // check for existing instance |
| 503 | fs = NULL; | 503 | fs = NULL; |
| 504 | my_debug("checking for existing PID file (%s)", cnf_pidfile); | 504 | my_debug("checking for existing PID file (%s)", cnf_pidfile); |
| 505 | if ((rc = stat(cnf_pidfile, &sb)) == -1) | 505 | if (stat(cnf_pidfile, &sb) == -1) |
| 506 | { | 506 | { |
| 507 | if (errno != ENOENT) | 507 | if (errno != ENOENT) |
| 508 | { | 508 | { |
| @@ -520,7 +520,7 @@ my_daemonize( | |||
| 520 | { | 520 | { |
| 521 | fscanf(fs, "%i", &pid); | 521 | fscanf(fs, "%i", &pid); |
| 522 | fclose(fs); | 522 | fclose(fs); |
| 523 | if ((rc = kill(pid, 0)) == -1) | 523 | if (kill(pid, 0) == -1) |
| 524 | { | 524 | { |
| 525 | if (errno == ESRCH) | 525 | if (errno == ESRCH) |
| 526 | { | 526 | { |
| @@ -548,7 +548,7 @@ my_daemonize( | |||
| 548 | return(-1); | 548 | return(-1); |
| 549 | }; | 549 | }; |
| 550 | my_debug("temp pidfile: %s", pidfile); | 550 | my_debug("temp pidfile: %s", pidfile); |
| 551 | if ((rc = link(pidfile, cnf_pidfile)) == -1) | 551 | if (link(pidfile, cnf_pidfile) == -1) |
| 552 | { | 552 | { |
| 553 | my_error("mkstemp(): %s", strerror(errno)); | 553 | my_error("mkstemp(): %s", strerror(errno)); |
| 554 | close(fd); | 554 | close(fd); |
| @@ -556,7 +556,7 @@ my_daemonize( | |||
| 556 | return(-1); | 556 | return(-1); |
| 557 | }; | 557 | }; |
| 558 | unlink(pidfile); | 558 | unlink(pidfile); |
| 559 | if ((rc = fchmod(fd, 0644)) == -1) | 559 | if (fchmod(fd, 0644) == -1) |
| 560 | { | 560 | { |
| 561 | my_error("fchmod(): %s", strerror(errno)); | 561 | my_error("fchmod(): %s", strerror(errno)); |
| 562 | close(fd); | 562 | close(fd); |
| @@ -565,7 +565,7 @@ my_daemonize( | |||
| 565 | if ( (cnf_uid != getuid()) || | 565 | if ( (cnf_uid != getuid()) || |
| 566 | (cnf_gid != getgid()) ) | 566 | (cnf_gid != getgid()) ) |
| 567 | { | 567 | { |
| 568 | if ((rc = fchown(fd, cnf_uid, cnf_gid)) == -1) | 568 | if (fchown(fd, cnf_uid, cnf_gid) == -1) |
| 569 | { | 569 | { |
| 570 | my_error("fchown(): %s", strerror(errno)); | 570 | my_error("fchown(): %s", strerror(errno)); |
| 571 | close(fd); | 571 | close(fd); |
| @@ -584,7 +584,7 @@ my_daemonize( | |||
| 584 | socklen = sizeof(struct sockaddr_in6); | 584 | socklen = sizeof(struct sockaddr_in6); |
| 585 | } else | 585 | } else |
| 586 | { | 586 | { |
| 587 | if ((rc = inet_pton(AF_INET, cnf_listen, &sa.sin.sin_addr)) == 1) | 587 | if (inet_pton(AF_INET, cnf_listen, &sa.sin.sin_addr) == 1) |
| 588 | { | 588 | { |
| 589 | sa.sin.sin_family = AF_INET; | 589 | sa.sin.sin_family = AF_INET; |
| 590 | sa.sin.sin_port = htons(cnf_port); | 590 | sa.sin.sin_port = htons(cnf_port); |
| @@ -619,7 +619,7 @@ my_daemonize( | |||
| 619 | // set socket options | 619 | // set socket options |
| 620 | my_debug("setting socket options"); | 620 | my_debug("setting socket options"); |
| 621 | opt = 1; | 621 | opt = 1; |
| 622 | if ((rc = setsockopt(s, SOL_SOCKET, SO_REUSEADDR, (void *)&opt, sizeof(int))) == -1) | 622 | if (setsockopt(s, SOL_SOCKET, SO_REUSEADDR, (void *)&opt, sizeof(int)) == -1) |
| 623 | { | 623 | { |
| 624 | my_error("setsockopt(SO_REUSEADDR): %s", strerror(errno)); | 624 | my_error("setsockopt(SO_REUSEADDR): %s", strerror(errno)); |
| 625 | close(s); | 625 | close(s); |
| @@ -630,7 +630,7 @@ my_daemonize( | |||
| 630 | 630 | ||
| 631 | // bind socket to interface | 631 | // bind socket to interface |
| 632 | my_debug("binding socket"); | 632 | my_debug("binding socket"); |
| 633 | if ((rc = bind(s, &sa.sa, socklen)) == -1) | 633 | if (bind(s, &sa.sa, socklen) == -1) |
| 634 | { | 634 | { |
| 635 | my_error("bind(): %s", strerror(errno)); | 635 | my_error("bind(): %s", strerror(errno)); |
| 636 | close(s); | 636 | close(s); |
| @@ -641,7 +641,7 @@ my_daemonize( | |||
| 641 | 641 | ||
| 642 | // log socket address | 642 | // log socket address |
| 643 | socklen = sizeof(struct sockaddr_storage); | 643 | socklen = sizeof(struct sockaddr_storage); |
| 644 | if ((rc = getsockname(s, &sa.sa, &socklen)) == -1) | 644 | if (getsockname(s, &sa.sa, &socklen) == -1) |
| 645 | { | 645 | { |
| 646 | my_error("getsockname(): %s", strerror(errno)); | 646 | my_error("getsockname(): %s", strerror(errno)); |
| 647 | close(s); | 647 | close(s); |
| @@ -670,7 +670,7 @@ my_daemonize( | |||
| 670 | }; | 670 | }; |
| 671 | 671 | ||
| 672 | // change ownership | 672 | // change ownership |
| 673 | if ( (getgid() != cnf_gid) && ((rc = setregid(cnf_gid, cnf_gid)) == -1) ) | 673 | if ( (getgid() != cnf_gid) && (setregid(cnf_gid, cnf_gid) == -1) ) |
| 674 | { | 674 | { |
| 675 | my_error("getgid(): %s", strerror(errno)); | 675 | my_error("getgid(): %s", strerror(errno)); |
| 676 | close(s); | 676 | close(s); |
| @@ -678,7 +678,7 @@ my_daemonize( | |||
| 678 | unlink(cnf_pidfile); | 678 | unlink(cnf_pidfile); |
| 679 | return(-1); | 679 | return(-1); |
| 680 | }; | 680 | }; |
| 681 | if ( (getuid() != cnf_uid) && ((rc = setreuid(cnf_uid, cnf_uid)) == -1) ) | 681 | if ( (getuid() != cnf_uid) && (setreuid(cnf_uid, cnf_uid) == -1) ) |
| 682 | { | 682 | { |
| 683 | my_error("getuid(): %s", strerror(errno)); | 683 | my_error("getuid(): %s", strerror(errno)); |
| 684 | close(s); | 684 | close(s); |
