[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-Id: <200811290048.47309.steffen.joeris@skolelinux.de>
Date: Sat, 29 Nov 2008 00:48:47 +0100
From: Steffen Joeris <steffen.joeris@...lelinux.de>
To: Jeremias Reith <jr@...ss.org>
Cc: oss-security@...ts.openwall.com
Subject: Re: CVE requset: WordPress XSS vulnerability in RSS Feed Generator
Hi Jeremias
On Fri, 28 Nov 2008 11:31:05 pm Jeremias Reith wrote:
> On Nov 28, 2008, at 22:39 , Steffen Joeris wrote:
> > Hi
> >
> >> a XSS vulnerability has been discovered in WordPress.
> >>
> >> Vendor info:
> >> http://wordpress.org/development/2008/11/wordpress-265/
> >>
> >> Detailed information:
> >> http://www.securityfocus.com/archive/1/498652 (Note: It should be
> >> "prior to 2.6.5" in the summary)
> >
> > I might be off here, but doesn't the patch[0] create another XSS by
> > removing
> > wp_specialchars?
> >
> > Cheers
> > Steffen
> >
> > [0]:
> > http://trac.wordpress.org/changeset?old_path=tags%2F2.6.3&old=&new_path=t
> >ags%2F2.6.5&new=
>
> Looks fine to me.
>
> You probably missed that the added clean_url() is applied on the
> entire URL instead of wp_specialchars() to REQUSET_URI.
Yeah you're right and it appears that clean_url takes care of all the bad
characters. However, I am still wondering why upstream doesn't use
htmlspecialchars(). :)
Cheers
Steffen
Download attachment "signature.asc " of type "application/pgp-signature" (198 bytes)
Powered by blists - more mailing lists
Please check out the
Open Source Software Security Wiki, which is counterpart to this
mailing list.
Confused about mailing lists and their use?
Read about mailing lists on Wikipedia
and check out these
guidelines on proper formatting of your messages.
