1.1 --- a/src/message.c	Mon Oct 27 16:21:27 2008 +0100
     1.2 +++ b/src/message.c	Mon Oct 27 16:23:10 2008 +0100
     1.3 @@ -18,193 +18,208 @@
     1.4  
     1.5  #include "masqmail.h"
     1.6  
     1.7 -message *create_message()
     1.8 +message*
     1.9 +create_message()
    1.10  {
    1.11 -  message *msg = (message *)g_malloc(sizeof(message));
    1.12 -  if(msg){
    1.13 -    memset(msg, 0, sizeof(message));
    1.14 -    msg->data_size = -1;
    1.15 -  }
    1.16 -  return msg;
    1.17 +	message *msg = (message *) g_malloc(sizeof(message));
    1.18 +	if (msg) {
    1.19 +		memset(msg, 0, sizeof(message));
    1.20 +		msg->data_size = -1;
    1.21 +	}
    1.22 +	return msg;
    1.23  }
    1.24  
    1.25 -gint msg_calc_size(message *msg, gboolean is_smtp)
    1.26 +gint
    1.27 +msg_calc_size(message * msg, gboolean is_smtp)
    1.28  {
    1.29 -  GList *node;
    1.30 -  gint l_cnt = 0, c_cnt = 0;
    1.31 +	GList *node;
    1.32 +	gint l_cnt = 0, c_cnt = 0;
    1.33  
    1.34 -  /* header size */
    1.35 -  if(msg->hdr_list){
    1.36 -    for(node = g_list_first(msg->hdr_list); node; node = g_list_next(node)){
    1.37 -      if(node->data){
    1.38 -	header *hdr = (header *)(node->data);
    1.39 -	if(hdr->header){
    1.40 -	  char *p = hdr->header;
    1.41 -	  while(*p){
    1.42 -	    if(*p++ == '\n') l_cnt++;
    1.43 -	    c_cnt++;
    1.44 -	  }
    1.45 +	/* header size */
    1.46 +	if (msg->hdr_list) {
    1.47 +		for (node = g_list_first(msg->hdr_list); node; node = g_list_next(node)) {
    1.48 +			if (node->data) {
    1.49 +				header *hdr = (header *) (node->data);
    1.50 +				if (hdr->header) {
    1.51 +					char *p = hdr->header;
    1.52 +					while (*p) {
    1.53 +						if (*p++ == '\n')
    1.54 +							l_cnt++;
    1.55 +						c_cnt++;
    1.56 +					}
    1.57 +				}
    1.58 +			}
    1.59 +		}
    1.60  	}
    1.61 -      }
    1.62 -    }
    1.63 -  }
    1.64  
    1.65 -  /* empty line separating headers from data: */
    1.66 -  c_cnt++;
    1.67 -  l_cnt++;
    1.68 +	/* empty line separating headers from data: */
    1.69 +	c_cnt++;
    1.70 +	l_cnt++;
    1.71  
    1.72 -  /* data size */
    1.73 -  if(msg->data_list){
    1.74 -    for(node = g_list_first(msg->data_list); node; node = g_list_next(node)){
    1.75 -      if(node->data){
    1.76 -	char *p = node->data;
    1.77 -	while(*p){
    1.78 -	  if(*p++ == '\n') l_cnt++;
    1.79 -	  c_cnt++;
    1.80 +	/* data size */
    1.81 +	if (msg->data_list) {
    1.82 +		for (node = g_list_first(msg->data_list); node; node = g_list_next(node)) {
    1.83 +			if (node->data) {
    1.84 +				char *p = node->data;
    1.85 +				while (*p) {
    1.86 +					if (*p++ == '\n')
    1.87 +						l_cnt++;
    1.88 +					c_cnt++;
    1.89 +				}
    1.90 +			}
    1.91 +		}
    1.92  	}
    1.93 -      }
    1.94 -    }
    1.95 -  }
    1.96  
    1.97 -  return is_smtp ? c_cnt + l_cnt : c_cnt;
    1.98 +	return is_smtp ? c_cnt + l_cnt : c_cnt;
    1.99  }
   1.100  
   1.101 -void msg_free_data(message *msg)
   1.102 +void
   1.103 +msg_free_data(message * msg)
   1.104  {
   1.105 -  GList *node;
   1.106 +	GList *node;
   1.107  
   1.108 -  if(msg->data_list){
   1.109 -    for(node = g_list_first(msg->data_list); node; node = g_list_next(node)){
   1.110 -      if(node->data)
   1.111 -	g_free(node->data);
   1.112 -    }
   1.113 -    g_list_free(msg->data_list);
   1.114 -    msg->data_list = NULL;
   1.115 -  }
   1.116 +	if (msg->data_list) {
   1.117 +		for (node = g_list_first(msg->data_list); node; node = g_list_next(node)) {
   1.118 +			if (node->data)
   1.119 +				g_free(node->data);
   1.120 +		}
   1.121 +		g_list_free(msg->data_list);
   1.122 +		msg->data_list = NULL;
   1.123 +	}
   1.124  }
   1.125  
   1.126 -void destroy_message(message *msg)
   1.127 +void
   1.128 +destroy_message(message * msg)
   1.129  {
   1.130 -  GList *node;
   1.131 +	GList *node;
   1.132  
   1.133 -  if(msg->uid) g_free(msg->uid);
   1.134 -  if(msg->ident) g_free(msg->ident);
   1.135 -  if(msg->return_path) g_free(msg->return_path);
   1.136 +	if (msg->uid)
   1.137 +		g_free(msg->uid);
   1.138 +	if (msg->ident)
   1.139 +		g_free(msg->ident);
   1.140 +	if (msg->return_path)
   1.141 +		g_free(msg->return_path);
   1.142  
   1.143 -  if(msg->rcpt_list){
   1.144 -    for(node = g_list_first(msg->rcpt_list); node; node = g_list_next(node)){
   1.145 -      if(node->data)
   1.146 -	g_free(node->data);
   1.147 -    }
   1.148 -    g_list_free(msg->rcpt_list);
   1.149 -  }
   1.150 -  if(msg->hdr_list){
   1.151 -    for(node = g_list_first(msg->hdr_list); node; node = g_list_next(node)){
   1.152 -      if(node->data){
   1.153 -	header *hdr = (header *)(node->data);
   1.154 -	if(hdr->header)
   1.155 -	  g_free(hdr->header);
   1.156 -	g_free(node->data);
   1.157 -      }
   1.158 -    }
   1.159 -    g_list_free(msg->hdr_list);
   1.160 -  }
   1.161 +	if (msg->rcpt_list) {
   1.162 +		for (node = g_list_first(msg->rcpt_list); node; node = g_list_next(node)) {
   1.163 +			if (node->data)
   1.164 +				g_free(node->data);
   1.165 +		}
   1.166 +		g_list_free(msg->rcpt_list);
   1.167 +	}
   1.168 +	if (msg->hdr_list) {
   1.169 +		for (node = g_list_first(msg->hdr_list); node; node = g_list_next(node)) {
   1.170 +			if (node->data) {
   1.171 +				header *hdr = (header *) (node->data);
   1.172 +				if (hdr->header)
   1.173 +					g_free(hdr->header);
   1.174 +				g_free(node->data);
   1.175 +			}
   1.176 +		}
   1.177 +		g_list_free(msg->hdr_list);
   1.178 +	}
   1.179  
   1.180 -  if(msg->full_sender_name)
   1.181 -    g_free(msg->full_sender_name);
   1.182 +	if (msg->full_sender_name)
   1.183 +		g_free(msg->full_sender_name);
   1.184  
   1.185 -  msg_free_data(msg);
   1.186 +	msg_free_data(msg);
   1.187  
   1.188 -  g_free(msg);
   1.189 +	g_free(msg);
   1.190  }
   1.191  
   1.192 -void destroy_msg_list(GList *msg_list)
   1.193 +void
   1.194 +destroy_msg_list(GList * msg_list)
   1.195  {
   1.196 -  GList *msg_node;
   1.197 +	GList *msg_node;
   1.198  
   1.199 -  foreach(msg_list, msg_node){
   1.200 -    message *msg = (message *)(msg_node->data);
   1.201 -    destroy_message(msg);
   1.202 -  }
   1.203 -  g_list_free(msg_list);
   1.204 +	foreach(msg_list, msg_node) {
   1.205 +		message *msg = (message *) (msg_node->data);
   1.206 +		destroy_message(msg);
   1.207 +	}
   1.208 +	g_list_free(msg_list);
   1.209  }
   1.210  
   1.211 -msg_out *create_msg_out(message *msg)
   1.212 +msg_out*
   1.213 +create_msg_out(message * msg)
   1.214  {
   1.215 -  msg_out *msgout = NULL;
   1.216 +	msg_out *msgout = NULL;
   1.217  
   1.218 -  msgout = g_malloc(sizeof(msg_out));
   1.219 -  if(msgout){
   1.220 -    msgout->msg = msg;
   1.221 -    msgout->return_path = NULL;
   1.222 -    msgout->rcpt_list = NULL;
   1.223 -    
   1.224 -    msgout->hdr_list = NULL;
   1.225 -    msgout->xtra_hdr_list = NULL;
   1.226 -  }
   1.227 -  return msgout;
   1.228 +	msgout = g_malloc(sizeof(msg_out));
   1.229 +	if (msgout) {
   1.230 +		msgout->msg = msg;
   1.231 +		msgout->return_path = NULL;
   1.232 +		msgout->rcpt_list = NULL;
   1.233 +
   1.234 +		msgout->hdr_list = NULL;
   1.235 +		msgout->xtra_hdr_list = NULL;
   1.236 +	}
   1.237 +	return msgout;
   1.238  }
   1.239  
   1.240 -msg_out *clone_msg_out(msg_out *msgout_orig)
   1.241 +msg_out*
   1.242 +clone_msg_out(msg_out * msgout_orig)
   1.243  {
   1.244 -  if(msgout_orig){
   1.245 -    msg_out *msgout = create_msg_out(msgout_orig->msg);
   1.246 -    if(msgout){
   1.247 -      msgout->msg = msgout_orig->msg;
   1.248 -      if(msgout_orig->return_path)
   1.249 -	msgout->return_path = copy_address(msgout_orig->return_path);
   1.250 -      if(msgout_orig->hdr_list)
   1.251 -	msgout->hdr_list = g_list_copy(msgout_orig->hdr_list);
   1.252 -      /* FIXME: if this lives longer than the original
   1.253 -	 and we access one of the xtra hdrs, we will segfault
   1.254 -	 or cause some weird bugs: */
   1.255 -      msgout->xtra_hdr_list = NULL;
   1.256 -      if(msgout_orig->rcpt_list)
   1.257 -	msgout->rcpt_list = g_list_copy(msgout_orig->rcpt_list);
   1.258 -    }
   1.259 -    return msgout;
   1.260 -  }
   1.261 -  return NULL;
   1.262 +	if (msgout_orig) {
   1.263 +		msg_out *msgout = create_msg_out(msgout_orig->msg);
   1.264 +		if (msgout) {
   1.265 +			msgout->msg = msgout_orig->msg;
   1.266 +			if (msgout_orig->return_path)
   1.267 +				msgout->return_path = copy_address(msgout_orig->return_path);
   1.268 +			if (msgout_orig->hdr_list)
   1.269 +				msgout->hdr_list = g_list_copy(msgout_orig->hdr_list);
   1.270 +			/* FIXME: if this lives longer than the original
   1.271 +			   and we access one of the xtra hdrs, we will segfault
   1.272 +			   or cause some weird bugs: */
   1.273 +			msgout->xtra_hdr_list = NULL;
   1.274 +			if (msgout_orig->rcpt_list)
   1.275 +				msgout->rcpt_list = g_list_copy(msgout_orig->rcpt_list);
   1.276 +		}
   1.277 +		return msgout;
   1.278 +	}
   1.279 +	return NULL;
   1.280  }
   1.281  
   1.282 -GList *create_msg_out_list(GList *msg_list)
   1.283 +GList*
   1.284 +create_msg_out_list(GList * msg_list)
   1.285  {
   1.286 -  GList *msgout_list = NULL;
   1.287 -  GList *msg_node;
   1.288 +	GList *msgout_list = NULL;
   1.289 +	GList *msg_node;
   1.290  
   1.291 -  foreach(msg_list, msg_node){
   1.292 -    message *msg = (message *)(msg_node->data);
   1.293 -    msgout_list = g_list_append(msgout_list, create_msg_out(msg));
   1.294 -  }
   1.295 -  return msgout_list;
   1.296 +	foreach(msg_list, msg_node) {
   1.297 +		message *msg = (message *) (msg_node->data);
   1.298 +		msgout_list = g_list_append(msgout_list, create_msg_out(msg));
   1.299 +	}
   1.300 +	return msgout_list;
   1.301  }
   1.302  
   1.303 -void destroy_msg_out(msg_out *msgout)
   1.304 +void
   1.305 +destroy_msg_out(msg_out * msgout)
   1.306  {
   1.307 -  if(msgout){
   1.308 -    if(msgout->return_path)
   1.309 -      destroy_address(msgout->return_path);
   1.310 -    if(msgout->hdr_list)
   1.311 -      g_list_free(msgout->hdr_list);
   1.312 -    if(msgout->xtra_hdr_list){
   1.313 -      GList *hdr_node;
   1.314 -      foreach(msgout->xtra_hdr_list, hdr_node){
   1.315 -	header *hdr = (header *)(hdr_node->data);
   1.316 -	destroy_header(hdr);
   1.317 -      }
   1.318 -      g_list_free(msgout->xtra_hdr_list);
   1.319 -    }
   1.320 -    g_free(msgout);
   1.321 -  }
   1.322 +	if (msgout) {
   1.323 +		if (msgout->return_path)
   1.324 +			destroy_address(msgout->return_path);
   1.325 +		if (msgout->hdr_list)
   1.326 +			g_list_free(msgout->hdr_list);
   1.327 +		if (msgout->xtra_hdr_list) {
   1.328 +			GList *hdr_node;
   1.329 +			foreach(msgout->xtra_hdr_list, hdr_node) {
   1.330 +				header *hdr = (header *) (hdr_node->data);
   1.331 +				destroy_header(hdr);
   1.332 +			}
   1.333 +			g_list_free(msgout->xtra_hdr_list);
   1.334 +		}
   1.335 +		g_free(msgout);
   1.336 +	}
   1.337  }
   1.338  
   1.339 -void destroy_msg_out_list(GList *msgout_list)
   1.340 +void
   1.341 +destroy_msg_out_list(GList * msgout_list)
   1.342  {
   1.343 -  GList *msgout_node;
   1.344 +	GList *msgout_node;
   1.345  
   1.346 -  foreach(msgout_list, msgout_node){
   1.347 -    msg_out *msgout = (msg_out *)(msgout_node->data);
   1.348 -    destroy_msg_out(msgout);
   1.349 -  }
   1.350 -  g_list_free(msgout_list);
   1.351 +	foreach(msgout_list, msgout_node) {
   1.352 +		msg_out *msgout = (msg_out *) (msgout_node->data);
   1.353 +		destroy_msg_out(msgout);
   1.354 +	}
   1.355 +	g_list_free(msgout_list);
   1.356  }