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