aboutsummaryrefslogtreecommitdiffstatshomepage
path: root/src
diff options
context:
space:
mode:
authorDavid M. Syzdek <david.syzdek@acsalaska.com>2022-07-21 23:32:29 -0800
committerDavid M. Syzdek <david.syzdek@acsalaska.com>2022-07-21 23:32:29 -0800
commit1610e3b15c634f5f70ad75a7711cb0b899c0e7a9 (patch)
tree3a9426042a6820ca40a28759032627def75b0951 /src
parent63c92e872110e37a6319566446bf02984210c79c (diff)
downloadakcom-udpecho-1610e3b15c634f5f70ad75a7711cb0b899c0e7a9.tar.gz
akcom-udpecho-1610e3b15c634f5f70ad75a7711cb0b899c0e7a9.tar.bz2
akcom-udpecho-1610e3b15c634f5f70ad75a7711cb0b899c0e7a9.tar.xz
akcom-udpecho-1610e3b15c634f5f70ad75a7711cb0b899c0e7a9.zip
akcom-udpechod: resolving xcode analyzer warnings
Diffstat (limited to 'src')
-rw-r--r--src/akcom-udpechod.c22
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);