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(a)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