[SSSD] [PATCH] fill_id() fix LE/BE issue with wrong data type

Jakub Hrozek jhrozek at redhat.com
Fri Jan 30 14:36:39 UTC 2015


On Thu, Jan 29, 2015 at 09:17:28PM +0100, Sumit Bose wrote:
> Hi,
> 
> we try to squeeze a 64bit type into a 32bit one which kind of works on
> little-endian because we checked that the value is not larger then
> UINT32_MAX before but fails on big-endian. This patch tries to fix it.
> 
> bye,
> Sumit

ACK for this patch because it fixes a bug, but we should check the usage
of ldb_msg_find_attr_as_uint64() more systematically. There's a couple
more places where I saw the same issue.

I filed https://fedorahosted.org/sssd/ticket/2578 to track this.



More information about the sssd-devel mailing list