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

Jan Zelený jzeleny at redhat.com
Fri Jun 8 12:45:40 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

Pushed to master

Jan
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 836 bytes
Desc: This is a digitally signed message part.
URL: <https://lists.fedorahosted.org/pipermail/sssd-devel/attachments/20120608/324a984a/attachment.sig>


More information about the sssd-devel mailing list