novabbs
/
rocksolid-light
3
0
Fork 1
Code Issues 22 Pull Requests Releases Wiki Activity

Fix checking for missing boundary line. Still more to improve for multipart messages.

This commit is contained in:
Retro_Guy 2024-07-27 06:03:17 -07:00
parent 9e6a344453
commit 797aa147a6
1 changed files with 8 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

10
Rocksolid_Light/rocksolid/lib/message.inc.php
View File

@ -49,12 +49,12 @@ function message_parse($rawmessage)
// We have multible bodies, so we split the message into its parts // We have multible bodies, so we split the message into its parts
$boundary = "--" . $message->header->content_type_boundary; $boundary = "--" . $message->header->content_type_boundary;
// lets find the first part // lets find the first part
while ($rawmessage[$i] != $boundary) { while ($rawmessage[$i] != $boundary) {
$i ++;
// Missing boundary line?
if(!$rawmessage[$i]) { if(!$rawmessage[$i]) {
break; break;
} }
$i ++;
} }
$i ++; $i ++;
$part = array(); $part = array();
@ -837,6 +837,12 @@ function message_show($group, $id, $attachment = 0, $article_data = false, $maxl
return "no-archive"; return "no-archive";
} }
if(isset($head->content_type[0])) {
if(!strpos($head->content_type[0], "/")) {
echo '<hr><p class=np_ob_posted_date>(message #' . $head->number . ' not displayed - malformed header)</p><hr>';
return "blocked";
}
}
if (($head->content_type[$attachment] == "text/plain") && ($attachment == 0)) { if (($head->content_type[$attachment] == "text/plain") && ($attachment == 0)) {
show_header($head, $group, $local_poster); show_header($head, $group, $local_poster);
// X-Face // X-Face