Opened 4 years ago

Last modified 3 years ago

#16508 new bug

Crash while changing ControlLook/Decorator

Reported by: bitigchi Owned by: nobody
Priority: normal Milestone: Unscheduled
Component: Preferences/Appearance Version: R1/beta2
Keywords: Cc:
Blocked By: Blocking:
Platform: All

Description

Crash report attached.

Attachments (1)

Appearance-386-debug-03-09-2020-19-38-47.report (8.9 KB ) - added by bitigchi 4 years ago.
crash report

Download all attachments as: .zip

Change History (3)

by bitigchi, 4 years ago

crash report

comment:1 by jackburton, 3 years ago

I think that commit b46615c55ad2c8fe6de54412055a0713da3d610a could be the culprit.

Shouldn't every reply start with StartMessage() ? In the app server, file ServerApp.cpp, lines starting from 663:

case AS_SET_DECORATOR:
{
	// Attached Data:
	// path to decorator add-on

	BString path;
	link.ReadString(path);

	status_t error = gDecorManager.SetDecorator(path, fDesktop);

	fLink.Attach<status_t>(error);
	fLink.Flush();

	if (error == B_OK)
		fDesktop->BroadcastToAllApps(AS_UPDATE_DECORATOR);
	break;

I think it's missing the fLink.StartMessage(error)

comment:2 by jackburton, 3 years ago

I mean:

diff --git a/src/servers/app/ServerApp.cpp b/src/servers/app/ServerApp.cpp index 198dc2fb87..660413f437 100644 --- a/src/servers/app/ServerApp.cpp +++ b/src/servers/app/ServerApp.cpp @@ -670,6 +670,7 @@ ServerApp::_DispatchMessage(int32 code, BPrivate::LinkReceiver& link)

status_t error = gDecorManager.SetDecorator(path, fDesktop);

+ fLink.StartMessage(error);

fLink.Attach<status_t>(error); fLink.Flush();

Version 0, edited 3 years ago by jackburton (next)
Note: See TracTickets for help on using tickets.