Opened 4 years ago

Closed 4 years ago

#1485653 closed Bugs (fixed)

Message with Emote-Icons doesn't work

Reported by: Hosteurope Owned by:
Priority: 1 - Highest Milestone: 0.2-stable
Component: PHP backend Version: 0.2-beta
Severity: critical Keywords: emote loop
Cc:

Description

If you send a HTML Message and include a Emote-Icon the send process will endless-loop.

This only happens if you INSTALL_PATH is longer than 42 characters.

This happens because there's a bug in the rcmail_attach_emoteicons function in program/steps/mail/sendmail.inc so there's an endless-lopp

to get the last replaced position in the while-loop you have to add the strlen of the INSTALLATION_PATH constant. If you don't do this you will start right in the middle of the new src="" string, and found the $searchstr again, in front of you search-cursor.

just replace the

$last_img_pos = $pos2;

with:

$last_img_pos = $pos2+strlen( INSTALL_PATH. '/' );

and everything works as it should :)

Change History (4)

comment:1 Changed 4 years ago by Hosteurope

  • Type changed from Patches to Bugs

comment:2 Changed 4 years ago by Hosteurope

  • Priority changed from 5 to 1 - Highest

comment:3 Changed 4 years ago by alec

  • Milestone changed from 0.2.1 to 0.2-stable

comment:4 Changed 4 years ago by alec

  • Component changed from MIME parsing to PHP backend
  • Resolution set to fixed
  • Status changed from new to closed

Fixed in [99f2b311].

Note: See TracTickets for help on using tickets.