X-Git-Url: https://code.citadel.org/?a=blobdiff_plain;f=webcit%2Fmsg_renderers.c;h=2be17adfda411f905633b8ed13a9f047647280c4;hb=64739000c20cd3605c6a68efbea7227796ee6009;hp=a58f64043a5dbb6aff7e6e5e56568c788fc6f0ed;hpb=ff7953d44ab38c3754bc2e1e9e4fa647c57c4029;p=citadel.git

diff --git a/webcit/msg_renderers.c b/webcit/msg_renderers.c
index a58f64043..2be17adfd 100644
--- a/webcit/msg_renderers.c
+++ b/webcit/msg_renderers.c
@@ -1275,12 +1275,15 @@ int ParseMessageListHeaders_Detail(StrBuf *Line,
 				   StrBuf *ConversionBuffer)
 {
 	wcsession *WCC = WC;
+	long len;
+	long totallen;
 
 	CheckConvertBufs(WCC);
 
-	Msg->from = NewStrBufPlain(NULL, StrLength(Line));
-	StrBufExtract_NextToken(ConversionBuffer, Line, pos, '|');
-	if (StrLength(ConversionBuffer) != 0) {
+	totallen = StrLength(Line);
+	Msg->from = NewStrBufPlain(NULL, totallen);
+	len = StrBufExtract_NextToken(ConversionBuffer, Line, pos, '|');
+	if (len > 0) {
 		/* Handle senders with RFC2047 encoding */
 		StrBuf_RFC822_2_Utf8(Msg->from, 
 				     ConversionBuffer, 
@@ -1291,9 +1294,9 @@ int ParseMessageListHeaders_Detail(StrBuf *Line,
 	}
 			
 	/* node name */
-	StrBufExtract_NextToken(ConversionBuffer, Line, pos, '|');
-	if ((StrLength(ConversionBuffer) !=0 ) &&
-		    ( ((WCC->CurRoom.QRFlags & QR_NETWORK)
+	len = StrBufExtract_NextToken(ConversionBuffer, Line, pos, '|');
+	if ((len > 0 ) &&
+	    ( ((WCC->CurRoom.QRFlags & QR_NETWORK)
 	       || ((strcasecmp(ChrPtr(ConversionBuffer), ChrPtr(WCC->serv_info->serv_nodename))
 		    && (strcasecmp(ChrPtr(ConversionBuffer), ChrPtr(WCC->serv_info->serv_fqdn))))))))
 	{
@@ -1305,7 +1308,7 @@ int ParseMessageListHeaders_Detail(StrBuf *Line,
 	 *	StrBufExtract_token(Msg->inetaddr, Line, 4, '|');
 	 */
 	StrBufSkip_NTokenS(Line, pos, '|', 1);
-	Msg->subj = NewStrBufPlain(NULL, StrLength(Line));
+	Msg->subj = NewStrBufPlain(NULL, totallen);
 
 	FlushStrBuf(ConversionBuffer);
 	/* we assume the subject is the last parameter inside of the list; 
@@ -1313,17 +1316,17 @@ int ParseMessageListHeaders_Detail(StrBuf *Line,
 	 * on tokenizer chars inside of the subjects
 	StrBufExtract_NextToken(ConversionBuffer,  Line, pos, '|');
 	*/
+	len = 0;
 	if (*pos != StrBufNOTNULL) {
-		StrBufPlain(ConversionBuffer, *pos, 
-			    StrLength(Line) - (*pos - ChrPtr(Line)));
+		len = totallen - (*pos - ChrPtr(Line));
+		StrBufPlain(ConversionBuffer, *pos, len);
 		*pos = StrBufNOTNULL;
-		if ((StrLength(ConversionBuffer) > 0) &&
-		    (*(ChrPtr(ConversionBuffer) + 
-		       StrLength(ConversionBuffer) - 1) == '|'))
+		if ((len > 0) &&
+		    (*(ChrPtr(ConversionBuffer) + len - 1) == '|'))
 			StrBufCutRight(ConversionBuffer, 1);
 	}
 
-	if (StrLength(ConversionBuffer) == 0)
+	if (len == 0)
 		StrBufAppendBufPlain(Msg->subj, _("(no subject)"), -1,0);
 	else {
 		StrBuf_RFC822_2_Utf8(Msg->subj,