Skip to content

/ rust-layers

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 crash from failing XOpenDisplay #38

Merged
merged 2 commits into from Nov 13, 2013
Merged

Fix crash from failing XOpenDisplay #38

merged 2 commits into from Nov 13, 2013
+7 −1

Commits on Nov 13, 2013

  1. Fix unsafe use of CString::with_ref 

    Sometimes the C string would get freed before XOpenDisplay was done with it,
producing a garbage display string and a null Display*.

Also switch to with_c_str because it will stack-allocate for the common case of
small strings (doesn't really matter here, but why not).
    @kmcallister
    kmcallister committed Nov 13, 2013
    7a70694

  2. Check for XOpenDisplay() failure

    @kmcallister
    kmcallister committed Nov 13, 2013
    456f154
You can’t perform that action at this time.