[Freeipa-devel] [PATCH] 061 Validate NAPTR records

[Rob Crittenden]

Jakub Hrozek wrote:
> On Mon, Feb 21, 2011 at 01:18:07PM -0500, Rob Crittenden wrote:
>> Jakub Hrozek wrote:
>>> -----BEGIN PGP SIGNED MESSAGE-----
>>> Hash: SHA1
>>>
>>> I'm not sure about checking the flags - this might be a little too much
>>> validation.
>>>
>>> https://fedorahosted.org/freeipa/ticket/840
>>
>> I think the flags length check needs to change. I would do this instead:
>>
>> flags = flags.replace('"','')
>>
>> Otherwise someone might try to pass in the flags 'SAU' and all that
>> would get set is A.
>>
>> rob
>
> OK, that's much better. New patch attached.

ack, pushed to master