ipc: use deleteLater() in IpcServerConnection to prevent use-after-free

src/ipc/ipc.cpp

@@ -61,7 +61,7 @@ IpcServerConnection::IpcServerConnection(QLocalSocket* socket, IpcServer* server
 
 void IpcServerConnection::onDisconnected() {
 	qCInfo(logIpc) << "IPC connection disconnected" << this;
-	delete this;
+	this->deleteLater();
 }
 
 void IpcServerConnection::onReadyRead() {
@@ -88,7 +88,7 @@ void IpcServerConnection::onReadyRead() {
 
 	// async connections reparent
 	if (dynamic_cast<IpcServer*>(this->parent()) != nullptr) {
-		delete this;
+		this->deleteLater();
 	}
 }
 

src/window/proxywindow.cpp

@@ -288,12 +288,18 @@ void ProxyWindowBase::setVisibleDirect(bool visible) {
 			this->bBackerVisibility = false;
 			this->deleteWindow();
 		}
-	} else if (this->window != nullptr) {
+	} else {
+		if (visible && this->window == nullptr) {
+			this->createWindow();
+		}
+
+		if (this->window != nullptr) {
 			if (visible) this->polishItems();
 			this->window->setVisible(visible);
 			this->bBackerVisibility = visible;
 		}
 	}
+}
 
 void ProxyWindowBase::schedulePolish() {
 	if (this->isVisibleDirect()) {