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
Pushed to master
Jan