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].