[SSSD] [PATCH] sudo: use dbus array for rules refresh

Jakub Hrozek jhrozek at redhat.com
Thu Jul 24 11:56:51 UTC 2014


On Thu, Jul 24, 2014 at 01:25:28PM +0200, Lukas Slebodnik wrote:
> On (23/07/14 14:30), Pavel Březina wrote:
> >https://fedorahosted.org/sssd/ticket/2387
> >
> >I wrote the code originally with very little understanding of D-Bus and its
> >quite awful. This is just a hotfix, but I think the code deserves some nice
> >refactoring.
> >
> >I'd like to use our new codegen tool and do it for all handlers at once. I
> >don't see any ticket for refactoring data provider code to use our new dbus
> >facility, should I create one or did I just miss it?
> 
> >From 6d361fab09fea7945621afe43f70aa18f32937e4 Mon Sep 17 00:00:00 2001
> >From: =?UTF-8?q?Pavel=20B=C5=99ezina?= <pbrezina at redhat.com>
> >Date: Wed, 23 Jul 2014 14:21:34 +0200
> >Subject: [PATCH] sudo: use dbus array for rules refresh
> >
> >D-Bus only supports 255 signatures which caused a segmentation fault
> >when sudo responder tried to refresh more rules at once.
> >
> >Resolves:
> >https://fedorahosted.org/sssd/ticket/2387
> >---
> 
> Tested with 500 sudo rules and small refrest timeouts. It works like a charm
> :-) Patch looks dood as well.
> 
> ACK
> 
> LS

* master:
    1d9b37ac83fdaf0b5daa70d957c23143312f949e
* sssd-1-11:
    dfef1d050c35398c6061256a947b4cc9c1f4b8e6



More information about the sssd-devel mailing list