[Freeipa-devel] [PATCH 0141] Generalize attribute_name<->rdata_type conversions.

Tomas Hozza thozza at redhat.com
Mon May 6 15:40:37 UTC 2013

On 04/08/2013 07:45 PM, Petr Spacek wrote:
> Hello,
> Generalize attribute_name<->rdata_type conversions.
> Attribute names are generated on-the-fly: String "Record" is appended
> to textual representation of DNS RDATA type.
> String "Record" is cut down from the attribute name during
> attribute name to rdata type conversion.
> From now, the plugin doesn't add artificial limitation to supported
> record types.


The patch looks good. (I didn't do functional test)

Cosmetic issue:
I think it would be good to dynamically allocate "mod_type" in LDAPMod
in every case and include the "mod_type" memory freeing in
free_ldapmod() function. Now one has to be be careful when it is
statically or dynamically allocated. Before it was static in every case.


Tomas Hozza

More information about the Freeipa-devel mailing list