Merge pull request 'this is actually a pipe not a file, it should be closed as a pipe' (#6) from themusicgod1/snac2:master into master

Reviewed-on: https://codeberg.org/grunfink/snac2/pulls/6

1 files changed, 3 insertions, 3 deletions

diff --git a/activitypub.c b/activitypub.c
index bb091ff..5f26f73 100644
--- a/activitypub.c
+++ b/activitypub.c
@@ -1017,10 +1017,10 @@ void process_queue(snac *snac)
             FILE *f;
             int ok = 0;
 
-            if ((f = popen("/usr/sbin/sendmail -t", "w")) != NULL) {
+            f = popen("/usr/sbin/sendmail -t", "w");
+            if (f) {
                 fprintf(f, "%s\n", msg);
-
-                if (fclose(f) != EOF)
+                if (pclose(f) != EOF) //this is a pipe stream not just a file
                     ok = 1;
             }