That was kind of my concern too.I can understand simple object
de-referencing taking place (e.g. person[1].name), but expression
evaluation (e.g. person[1].gobbleUpMemory()+person[2].gobbleUpMemory())
does seem potentially dangerous.
Al.
Chris Pratt wrote:
I've seen it as well on 2.1.2 since I upgraded DisplayTag to 1.2. It
appears to log the exception and continue, but my concern would be we have
another vector for attack. Knowing that the system is going to evaluate the
name of, at least, GET variables using OGNL, it would probably be possible
to inject some OGNL that will compromise the security of sites built on the
framework.
(*Chris*)
On Tue, Dec 30, 2008 at 6:16 AM, Musachy Barroso <musa...@gmail.com> wrote:
Does it just log the exception and keeps going, or it breaks? Can you
post the stacktrace? AFAIK the parameters are always evaluated,
otherwise params like person[1].name would never be populated.
musachy
On Tue, Dec 30, 2008 at 4:12 AM, Al Sutton <a...@funkyandroid.com> wrote:
The previous problems are fixed, but there looks like there might be a
potential issue with XWork and/or OGNL....
I use displaytag and when paging through the URL is something like;
http://localhost/AndroidPhoneApplications/subs/?d-16387-p=3
When paging through with 2.1.4 I get an exception from XWork/OGNL which
says;
ognl.InappropriateExpressionException: Inappropriate OGNL expression: (d
-
16387) - p
I don't get this with 2.1.2, so is attempts to evaluate GET parameter
names
a new thing with 2.1.4?
imho Xwork/OGNL shouldn't be evaluating either the name or the value of a
POST or GET parameter because they are user submitted and there maybe
something which the application needs to process but is in an form that
OGNL
"changes", and hence a lot of confusion could be caused.
Al.
P.S. I don't know if it's happening with POST parameters as well.
--
======
Funky Android Limited is registered in England & Wales with the company
number 6741909. The registered head office is Kemp House, 152-160 City
Road, London, EC1V 2NX, UK.
The views expressed in this email are those of the author and not
necessarily those of Funky Android Limited, it's associates, or it's
subsidiaries.
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscr...@struts.apache.org
For additional commands, e-mail: dev-h...@struts.apache.org
--
"Hey you! Would you help me to carry the stone?" Pink Floyd
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscr...@struts.apache.org
For additional commands, e-mail: dev-h...@struts.apache.org
--
======
Funky Android Limited is registered in England & Wales with the
company number 6741909. The registered head office is Kemp House,
152-160 City Road, London, EC1V 2NX, UK.
The views expressed in this email are those of the author and not
necessarily those of Funky Android Limited, it's associates, or it's
subsidiaries.
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscr...@struts.apache.org
For additional commands, e-mail: dev-h...@struts.apache.org
