[SSSD] [PATCH] SUDO: Provide a sudo DP request based on the internal_req

Jakub Hrozek jhrozek at redhat.com
Mon Jan 23 15:08:13 UTC 2012


On Mon, Jan 23, 2012 at 03:55:26PM +0100, Pavel Březina wrote:
> Dne 23.1.2012 15:39, Jakub Hrozek napsal(a):
> >On Mon, Jan 23, 2012 at 10:23:36AM +0100, Pavel Březina wrote:
> >>Dne 23.1.2012 02:24, Stephen Gallagher napsal(a):
> >>>Also, when you call sss_dp_issue_request(), you're passing 'state' as
> >>>the pvt data for sss_dp_get_sudoers_msg(), but you meant to pass 'info',
> >>>I think.
> >>
> >>Shouldn't we start using tallog_get_type_abort() to detect these situations?
> >>
> >
> >I'm not sure we should ever abort, except for situations like options
> >mismatch (those can be potentially very dangerous).
> 
> We can set up our own 'abort' function using talloc_set_abort_fn().
> For example: we can create an abort function that would kill sssd if
> it is under development but prints DEBUG messages in users version.
> 

I like this idea if we can get around the requirement of having to
support an old talloc version.

> This function is not yet documented.

You can send a patch to man talloc :-)



More information about the sssd-devel mailing list