Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 150180 - Drupal 4.7.3 <= vuln. characters injection
Summary: Drupal 4.7.3 <= vuln. characters injection
Status: RESOLVED WONTFIX
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: New packages (show other bugs)
Hardware: All Linux
: High minor
Assignee: Roy Marples (RETIRED)
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2006-10-05 10:08 UTC by Emanuele Gentili
Modified: 2007-01-03 03:06 UTC (History)
2 users (show)

See Also:
Package list:
Runtime testing required: ---


Attachments
remove msg drupal bug (remove_msg.patch,820 bytes, patch)
2006-10-05 15:19 UTC, Emanuele Gentili
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Emanuele Gentili 2006-10-05 10:08:57 UTC
In Drupal there is a little problem that originates characters injection.

http://www.website.dom/drupal-path/?q=user/MESSAGES

You can edit "messages" and insert what you prefer; 
Variable print it and show the messages in the page.

Drupal demo vulnerable too.

test: http://demo.opensourcecms.com/drupal/?q=user/messages


I will contact Drupal developers.
Comment 1 Chris White (RETIRED) gentoo-dev 2006-10-05 10:16:29 UTC
I'm a little confused on the security impact of this.  I tried it myself and it seems you have to be logged in to access that page anyways.  Because opensource cms is a single user, I wasn't able to check and see if I could access other users pages.
Comment 2 Tavis Ormandy (RETIRED) gentoo-dev 2006-10-05 10:21:29 UTC
guys, seriously, this is not a security issue.

Thanks for playing around and looking for problems, but "character injection" is not a vulnerability, web-apps are designed to work like tihs. If you can inject **arbitrary** characters, let us know.

RESOLVED -> INVALID.
Comment 3 Emanuele Gentili 2006-10-05 15:15:23 UTC
I talked with Heinet (Drupal security devel), and he patch it.
Comment 4 Emanuele Gentili 2006-10-05 15:16:22 UTC
Patch:

Simply remove line 902 - 904 from modules/user.module

if ($msg) {
$form['message'] = array('#value' => '<p>'. check_plain($msg) .'</p>');
}
Comment 5 Emanuele Gentili 2006-10-05 15:19:41 UTC
Created attachment 98887 [details, diff]
remove msg drupal bug
Comment 6 Luca Longinotti (RETIRED) gentoo-dev 2006-10-05 15:24:34 UTC
Reopening to reassign, sorry for the noise!
Best regards, CHTEKK.
Comment 7 Luca Longinotti (RETIRED) gentoo-dev 2006-10-05 15:25:26 UTC
Reassigning...
Comment 8 Heine 2006-10-05 15:31:21 UTC
Note; while I provided a patch (was asked for); we do not consider this a security issue.

Heine
Comment 9 Roy Marples (RETIRED) gentoo-dev 2007-01-03 03:06:10 UTC
No security issue, no need to fix then as I hate bumping web-apps needlessly

Drupal-4.7.4 has since come out without this patch in.