Ticket #592 (closed defect: fixed)

Opened 2 years ago

Last modified 2 years ago

http://bbpress.org/forums/bb-edit.php improperly filtered input

Reported by: chrishajer Assigned to:
Priority: high Milestone:
Component: Back-end Version: 0.8
Severity: major Keywords:
Cc:

Description

I was adding a new topic at bbpress.org/forums and used the <pre> tag in my post (which is not allowed, but blockquote does not set the text apart at all.) It resulted in a MySQL error that appears to be unfiltered input or a possible SQL injection.

Here are the steps.

1. I started a new topic.

2. Went to edit the post because I did not like the way the <blockquote> was styled (i.e. not indented) so I tried the <pre> tag. Not sure if I can add attachments here, but if I can, it is called post-submission-bbpress.txt. That is the text that was submitted after editing.

3. I got the attached error after editing that post: a. error-text-bbpress.txt is a copy of the text displayed in the browser b. error-source-bbpress.txt is the source of the above page c. sql-error-bbpress.png is a screenshot of the browser window without the chrome

I did not try to exploit it further. I know just enough about SQL injection to know that this shouldn't happen :)

Attachments

post-submission-bbpress.txt (0.7 kB) - added by chrishajer on 02/18/07 14:46:02.
error-text-bbpress.txt (0.9 kB) - added by chrishajer on 02/18/07 14:46:51.
error-source-bbpress.txt (1.2 kB) - added by chrishajer on 02/18/07 14:47:16.
sql-error-bbbpress.png (12.6 kB) - added by chrishajer on 02/18/07 14:48:08.

Change History

02/18/07 14:46:02 changed by chrishajer

  • attachment post-submission-bbpress.txt added.

02/18/07 14:46:51 changed by chrishajer

  • attachment error-text-bbpress.txt added.

02/18/07 14:47:16 changed by chrishajer

  • attachment error-source-bbpress.txt added.

02/18/07 14:48:08 changed by chrishajer

  • attachment sql-error-bbbpress.png added.

02/18/07 14:49:00 changed by chrishajer

  • owner changed.
  • component changed from Front-end to Back-end.

02/20/07 20:36:31 changed by mdawaffe

  • status changed from new to closed.
  • resolution set to fixed.

(In [717]) quircky slashes bug. Fixes #592