Segmentation fault auf XQueryTree

Question

Ich versuche, das zuletzt benutzte Fenster (das Fenster direkt unter dem aktuellen Fenster in der Stapelreihenfolge) zu schließen. Leider segmentiert XQueryTree aus irgendeinem Grund.

#pragma once 

#include <X11/Xlib.h> 
#include <X11/Xutil.h> 

namespace WindowingOperations { 

    inline void closeLastWindow() { 
     Display* dpy = XOpenDisplay(0); 
     Window root = DefaultRootWindow(dpy); 

     Window* root_return; 
     Window* parent_return; 
     Window** children_return; 
     unsigned int* nchildren_return; 

     XQueryTree(dpy, 
        root, 
        root_return, 
        parent_return, 
        children_return, 
        nchildren_return); 

     // Kill the window right after this one 
     if (*nchildren_return > 1) 
      XDestroyWindow(dpy, *children_return[*nchildren_return - 2]); 
    } 
} 

EDIT:

Wenn Sie einen Testfall benötigen:

#include "window_operations.h" 
int main() { 
    WindowingOperations::closeLastWindow(); 
    return 0; 
} 

Answer 1

Die _return Parameter müssen irgendwo gehen. Sie können nicht nur in nicht initialisierten Zeiger passieren, Speicher für XQueryTree zugewiesen werden muss, um die Ergebnisse zu schreiben.

So ...

namespace WindowingOperations { 

    inline void closeLastWindow() { 
     Display* dpy = XOpenDisplay(0); 
     Window root = DefaultRootWindow(dpy); 

    // Allocate storage for the results of XQueryTree. 
     Window root_return; 
     Window parent_return; 
     Window* children_return; 
     unsigned int nchildren_return; 

    // then make the call providing the addresses of the out parameters 
     if (XQueryTree(dpy, 
         root, 
         &root_return, 
         &parent_return, 
         &children_return, 
         &nchildren_return) != 0) 
     { // added if to test for a failed call. results are unchanged if call failed, 
      // so don't use them 

      // Kill the window right after this one 
      if (*nchildren_return > 1) 
       XDestroyWindow(dpy, *children_return[*nchildren_return - 2]); 
     } 
     else 
     { 
      // handle error 
     } 
    } 
}