X-Git-Url: https://code.citadel.org/?a=blobdiff_plain;f=citadel%2Fmodules%2Finetcfg%2Fserv_inetcfg.c;h=c0ddb356669726b8f0b1adce5e8b3ba88fc56441;hb=d4d8a13594b84a4a051066cd6c9856d194f63783;hp=c37dd24fbdfcb103ea033bbb10fb5bdd2664b07c;hpb=158e8958500476d1115c7b70f6c81a87bb0ade47;p=citadel.git

diff --git a/citadel/modules/inetcfg/serv_inetcfg.c b/citadel/modules/inetcfg/serv_inetcfg.c
index c37dd24fb..c0ddb3566 100644
--- a/citadel/modules/inetcfg/serv_inetcfg.c
+++ b/citadel/modules/inetcfg/serv_inetcfg.c
@@ -4,21 +4,15 @@
  * wrote it as a module merely to keep things as clean and loosely coupled
  * as possible.
  *
- * Copyright (c) 1987-2012 by the citadel.org team
+ * Copyright (c) 1987-2017 by the citadel.org team
  *
- *  This program is open source software; you can redistribute it and/or modify
- *  it under the terms of the GNU General Public License version 3.
- *  
- *  
+ * This program is open source software; you can redistribute it and/or modify
+ * it under the terms of the GNU General Public License version 3.
  *
- *  This program is distributed in the hope that it will be useful,
- *  but WITHOUT ANY WARRANTY; without even the implied warranty of
- *  MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
- *  GNU General Public License for more details.
- *
- *  
- *  
- *  
+ * This program is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+ * GNU General Public License for more details.
  */
 
 #include "sysdep.h"
@@ -57,8 +51,6 @@
 #include "internet_addressing.h"
 #include "genstamp.h"
 #include "domain.h"
-
-
 #include "ctdl_module.h"
 
 
@@ -92,18 +84,23 @@ int inetcfg_aftersave(struct CtdlMessage *msg, recptypes *recp) {
 	/* If this isn't the configuration room, or if this isn't a MIME
 	 * message, don't bother.
 	 */
-	if (strcasecmp(msg->cm_fields[eOriginalRoom], SYSCONFIGROOM)) return(0);
-	if (msg->cm_format_type != 4) return(0);
+	if ((msg->cm_fields[eOriginalRoom]) && (strcasecmp(msg->cm_fields[eOriginalRoom], SYSCONFIGROOM))) {
+		return(0);
+	}
+	if (msg->cm_format_type != 4) {
+		return(0);
+	}
 
 	ptr = msg->cm_fields[eMesageText];
 	while (ptr != NULL) {
 	
 		linelen = strcspn(ptr, "\n");
-		if (linelen == 0) return(0);	/* end of headers */	
+		if (linelen == 0) {
+			return(0);	/* end of headers */	
+		}
 		
 		if (!strncasecmp(ptr, "Content-type: ", 14)) {
-			if (!strncasecmp(&ptr[14], INTERNETCFG,
-		   	   strlen(INTERNETCFG))) {
+			if (!strncasecmp(&ptr[14], INTERNETCFG, strlen(INTERNETCFG))) {
 				inetcfg_setTo(msg);	/* changing configs */
 			}
 		}
@@ -119,7 +116,7 @@ int inetcfg_aftersave(struct CtdlMessage *msg, recptypes *recp) {
 void inetcfg_init_backend(long msgnum, void *userdata) {
 	struct CtdlMessage *msg;
 
-       	msg = CtdlFetchMessage(msgnum, 1);
+       	msg = CtdlFetchMessage(msgnum, 1, 1);
        	if (msg != NULL) {
 		inetcfg_setTo(msg);
                	CM_Free(msg);
@@ -128,37 +125,36 @@ void inetcfg_init_backend(long msgnum, void *userdata) {
 
 
 void inetcfg_init(void) {
-	if (CtdlGetRoom(&CC->room, SYSCONFIGROOM) != 0) return;
-	CtdlForEachMessage(MSGS_LAST, 1, NULL, INTERNETCFG, NULL,
-		inetcfg_init_backend, NULL);
+	if (CtdlGetRoom(&CC->room, SYSCONFIGROOM) != 0) {
+		return;
+	}
+	CtdlForEachMessage(MSGS_LAST, 1, NULL, INTERNETCFG, NULL, inetcfg_init_backend, NULL);
 }
 
 
-
-
 /*****************************************************************************/
 /*                      MODULE INITIALIZATION STUFF                          */
 /*****************************************************************************/
-void clenaup_inetcfg(void)
+void cleanup_inetcfg(void)
 {
 	char *buf;
-
 	buf = inetcfg;
 	inetcfg = NULL;
-	if (buf != NULL)
+	if (buf != NULL) {
 		free(buf);
+	}
 }
 
+
 CTDL_MODULE_INIT(inetcfg)
 {
 	if (!threading)
 	{
 		CtdlRegisterMessageHook(inetcfg_aftersave, EVT_AFTERSAVE);
 		inetcfg_init();
-		CtdlRegisterCleanupHook(clenaup_inetcfg);
+		CtdlRegisterCleanupHook(cleanup_inetcfg);
 	}
 	
 	/* return our module name for the log */
 	return "inetcfg";
 }
-