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