more places where we can use cm_lengths;

[citadel.git] / citadel / modules / extnotify / funambol65.c

diff --git a/citadel/modules/extnotify/funambol65.c b/citadel/modules/extnotify/funambol65.c
index f4b0d8d38fc9b8943de839e903e1d08e9773c2bf..20246c90d02a38ac25193743a01440340b5fca48 100644 (file)
--- a/citadel/modules/extnotify/funambol65.c
+++ b/citadel/modules/extnotify/funambol65.c
@@ -36,6 +36,7 @@
 #include <curl/curl.h>
 
 #include "citadel.h"
+#include "citserver.h"
 #include "citadel_dirs.h"
 #include "clientsocket.h"
 #include "sysdep.h"
@@ -93,11 +94,15 @@ int notify_http_server(char *remoteurl,
 
        if (tlen > 0) {
                /* Load the template message. Get mallocs done too */
-               FILE *Ftemplate = NULL;
+               int fd;
+               struct stat statbuf;
                const char *mimetype;
+               const char *Err = NULL;
 
-               Ftemplate = fopen(template, "r");
-               if (Ftemplate == NULL) {
+               fd = open(template, O_RDONLY);
+               if ((fd < 0) ||
+                   (fstat(fd, &statbuf) == -1))
+               {
                        char buf[SIZ];
 
                        snprintf(buf, SIZ,
@@ -110,34 +115,35 @@ int notify_http_server(char *remoteurl,
                        CtdlAideMessage(
                                buf,
                                "External notifier: "
-                               "unable to find message template!");
+                               "unable to find/stat message template!");
                        goto abort;
                }
-               mimetype = GuessMimeByFilename(template, tlen);
-
-               buf = malloc(SIZ);
-               memset(buf, 0, SIZ);
-               SOAPMessage = malloc(3072);
-               memset(SOAPMessage, 0, 3072);
 
-               while(fgets(buf, SIZ, Ftemplate) != NULL) {
-                       strcat(SOAPMessage, buf);
-               }
-               fclose(Ftemplate);
-
-               if (strlen(SOAPMessage) < 0) {
+               Buf = NewStrBufPlain(NULL, statbuf.st_size + 1);
+               if (StrBufReadBLOB(Buf, &fd, 1, statbuf.st_size, &Err) < 0) {
                        char buf[SIZ];
 
+                       close(fd);
+
                        snprintf(buf, SIZ,
-                                "Cannot load template file %s;"
-                                " won't send notification\r\n",
-                                file_funambol_msg);
+                                "Cannot load template file %s [%s] "
+                                "won't send notification\r\n",
+                                file_funambol_msg,
+                                Err);
                        syslog(LOG_ERR, "%s", buf);
-
-                       CtdlAideMessage(buf, "External notifier: "
-                                       "unable to load message template!");
+                       // TODO: once an hour!
+                       CtdlAideMessage(
+                               buf,
+                               "External notifier: "
+                               "unable to load message template!");
                        goto abort;
                }
+               close(fd);
+
+               mimetype = GuessMimeByFilename(template, tlen);
+
+               SOAPMessage = SmashStrBuf(&Buf);
+
                // Do substitutions
                help_subst(SOAPMessage, "^notifyuser", user);
                help_subst(SOAPMessage, "^syncsource",
@@ -159,7 +165,8 @@ int notify_http_server(char *remoteurl,
                IO->HttpReq.headers = curl_slist_append(
                        IO->HttpReq.headers,
                        contenttype);
-
+               free(contenttype);
+               contenttype = NULL;
                IO->HttpReq.headers = curl_slist_append(
                        IO->HttpReq.headers,
                        "Accept: application/soap+xml, "
@@ -258,6 +265,7 @@ eNextState EvaluateResult(AsyncIO *IO)
                CtdlAideMessage(ChrPtr(ErrMsg),
                                "External notifier: "
                                "unable to contact notification host!");
+               FreeStrBuf(&ErrMsg);
        }
 
        syslog(LOG_DEBUG, "Funambol notified\n");
@@ -294,7 +302,7 @@ eNextState EvaluateResult(AsyncIO *IO)
        ///if (SOAPMessage != NULL) free(SOAPMessage);
        ///if (buf != NULL) free(buf);
        ///FreeStrBuf (&ReplyBuf);
-       return 0;
+       return eTerminateConnection;
 }
 
 eNextState ExtNotifyTerminateDB(AsyncIO *IO)