Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix a segfault in IFP's GetUserAttr #124

Closed
wants to merge 1 commit into from
Closed

Conversation

jhrozek
Copy link
Contributor

@jhrozek jhrozek commented Jan 15, 2017

This PR fixes a crash in GetUserAttr. To reproduce, it's enough to:
dbus-send --print-reply --system --dest=org.freedesktop.sssd.infopipe /org/freedesktop/sssd/infopipe org.freedesktop.sssd.infopipe.GetUserAttr string:admin array:string:name

GetUserAttr used to segfault without this patch.
@fidencio
Copy link
Contributor

Ack!

@lslebodn
Copy link
Contributor

lslebodn commented Jan 16, 2017

Do you know when the regression was created?
Was it caused by cache-req refactoring?

@fidencio
Copy link
Contributor

@lslebodn,

The issue was introduced by the commit b206e1abb7f6ea373d12537b3338552aed6b656d.
So, yes, it was introduced by the cache-req refactoring.

@lslebodn
Copy link
Contributor

lslebodn commented Jan 16, 2017 via email

@lslebodn lslebodn closed this Jan 16, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants