no longer free() a title pointer that wasn't malloced, but instead came from the outside (right from the middle of some port link stuff)

git-svn-id: file:///srv/svn/repos/haiku/haiku/trunk@14605 a95241bf-73f2-0310-859d-f6bbb57e9c96

src/servers/app/ServerWindow.cpp

@@ -131,7 +131,7 @@ struct dw_sync_data {
 ServerWindow::ServerWindow(const char *title, ServerApp *app,
 						   port_id clientPort, port_id looperPort, int32 handlerID)
 	: MessageLooper(title && *title ? title : "Unnamed Window"),
-	fTitle(title),
+	fTitle(strdup(title && *title ? title : "Unnamed Window")),
 	fDesktop(app->GetDesktop()),
 	fServerApp(app),
 	fWinBorder(NULL),
@@ -160,7 +160,7 @@ ServerWindow::~ServerWindow()
 
 	delete fWinBorder;
 
-	free(const_cast<char *>(fTitle));
+	free(fTitle);
 	delete_port(fMessagePort);
 
 	BPrivate::gDefaultTokens.RemoveToken(fServerToken);
@@ -173,11 +173,6 @@ ServerWindow::~ServerWindow()
 status_t
 ServerWindow::Init(BRect frame, uint32 look, uint32 feel, uint32 flags, uint32 workspace)
 {
-	if (fTitle == NULL)
-		fTitle = strdup("Unnamed Window");
-	if (fTitle == NULL)
-		return B_NO_MEMORY;
-
 	// fMessagePort is the port to which the app sends messages for the server
 	fMessagePort = create_port(100, fTitle);
 	if (fMessagePort < B_OK)
@@ -304,19 +299,12 @@ ServerWindow::Hide()
 void
 ServerWindow::SetTitle(const char* newTitle)
 {
-	const char* oldTitle = fTitle;
+	char *title = strdup(newTitle && *newTitle ? newTitle : "Unnamed Window");
-
+	if (!title)
-	if (newTitle == NULL || !newTitle[0])
-		fTitle = strdup("Unnamed Window");
-	else
-		fTitle = strdup(newTitle);
-
-	if (fTitle == NULL) {
-		fTitle = oldTitle;
 		return;
-	}
+	
-
+	free(fTitle);
-	free(const_cast<char*>(oldTitle));
+	fTitle = title;
 
 	if (Thread() >= B_OK) {
 		char name[B_OS_NAME_LENGTH];

src/servers/app/ServerWindow.h

@@ -128,7 +128,7 @@ private:
 												BPoint where);
 
 private:
-			const char*			fTitle;
+			char *				fTitle;
 
 			Desktop*			fDesktop;
 			ServerApp*			fServerApp;