[SSSD] [PATCH] sss_idmap: fix typo which prevents sub auth larger then 2^31

Jakub Hrozek jhrozek at redhat.com
Wed Jun 6 07:40:34 UTC 2012


On Mon, Jun 04, 2012 at 05:11:51PM +0200, Sumit Bose wrote:
> On Mon, Jun 04, 2012 at 02:04:38PM +0200, Jakub Hrozek wrote:
> > On Fri, Jun 01, 2012 at 01:10:32PM +0200, Sumit Bose wrote:
> > > Hi,
> > > 
> > > I found a missing 'u' in the idmap code. This patch fixes the typo and
> > > adds a test.
> > > 
> > > bye,
> > > Sumit
> > 
> > The patch seems to be breaking unit tests:
> > 
> > Running suite(s): IDMAP
> > Leak report for src/tests/common_check.c:104:
> > full talloc report on 'talloc_new: src/tests/common_check.c:96' (total    216 bytes in   4 blocks)
> >     src/tests/sss_idmap-tests.c:50 contains     68 bytes in   1 blocks (ref 0) 0x13781d0
> >     src/tests/sss_idmap-tests.c:50 contains     80 bytes in   1 blocks (ref 0) 0x1378120
> >     src/tests/sss_idmap-tests.c:50 contains     68 bytes in   1 blocks (ref 0) 0x1378080
> > /bin/sh: line 5:  3702 Segmentation fault      (core dumped) LDB_MODULES_PATH=/home/jakub/devel/sssd/ldb_mod_test_dir ${dir}$tst
> > FAIL: sss_idmap-tests
> 
> new version attached.
> 
> Thank you for the review.
> 
> bye,
> Sumit

Ack



More information about the sssd-devel mailing list