New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
security Issues with user-defined \p{} properties #11063
Comments
From @iabynCreated by @iabynThis is a placeholder for the issues related to \p{UserDefined} being <4CD4336F.9000801@khwilliamson.com> Perl Info
|
From @iabynOn Sat, Jan 22, 2011 at 07:12:51AM -0800, Dave Mitchell wrote:
I have now added a taint check for the property function name with commit This means we now have the following new restrictions: * only call the function if its name begins with In or Is There doesn't seem to be any consensus yet on what other measures, if any, -- |
From @khwilliamsonOn 02/22/2011 09:37 AM, Dave Mitchell wrote:
I'm very glad this got done. I suspect this needs to be applied as well |
The RT System itself - Status changed from 'new' to 'open' |
From @iabynOn Tue, Feb 22, 2011 at 10:23:56AM -0700, Karl Williamson wrote:
I don't think the same rationale applies. The issue with \p{} was that -- |
From marvin@rectangular.comOn Tue, Feb 22, 2011 at 10:23:56AM -0700, Karl Williamson wrote:
Seconded. Thanks for taking this on, Dave! Marvin Humphrey |
From tchrist@perl.comKarl Williamson <public@khwilliamson.com> wrote
You mean for something that \U etc map to?
I don't understand the need for the function to match /^I[ns]/. I'd imagine a clearer registration system à la charnames might be # In case customizing charnames files are in bin not lib... use charnames ( I also expose those to patterns read in from the user, Why shouldn't we do custom properties in some similar fashion? --tom |
From @iabynOn Tue, Feb 22, 2011 at 11:02:53AM -0800, tchrist1 via RT wrote:
Because it was documented that it only called functions whose names
But we need something that does doesn't break existing code (too much) -- |
From tchrist@perl.com
Ok.
I see you recognize how those two goals can easily conflict. :) I believe Abigail has used custom properties a great deal more than I never use the /^I[sn]/. I guess I've come to dislike those particular I rather like \p{OUR::Customized} for a customization function in --tom |
From tchrist@perl.comI miswrote:
I of course meant: I still use \p{Scriptname} for \p{Script=Scriptname}, --tom |
From @AbigailOn Tue, Feb 22, 2011 at 12:18:54PM -0700, Tom Christiansen wrote:
Yeah. OTOH, I kind of like the idea that \p{Whatever} is possible. The I wonder if it's worthwhile to allow \p{Whatever} if 'use re "eval";' is
Outside of the realm "what happens if I pull this lever", I've only used Of course, most of the time, I use /[$whatever]/, or /$thisaswell/, because $whatever = "acegikmoqsuwy"; # or is a lot easier to write, and a lot clearer than subs for \p{IsWhatever}.
Abigail |
The case changing functions have been removed, and the restriction to In... Is... name, and the taintedness checking should be sufficient |
Migrated from rt.perl.org#82616 (status was 'open')
Searchable as RT82616$
The text was updated successfully, but these errors were encountered: