[bug #24200] NSSavePanel crashes apps

[Paul D. Smith]

URL:
  <http://savannah.gnu.org/bugs/?24200>

                 Summary: NSSavePanel crashes apps
                 Project: GNUstep
            Submitted by: thebeing
            Submitted on: Do 04 Sep 2008 10:04:10 GMT
                Category: Gui/AppKit
                Severity: 3 - Normal
              Item Group: Bug
                  Status: None
                 Privacy: Public
             Assigned to: None
             Open/Closed: Open
         Discussion Lock: Any

    _______________________________________________________

Details:

Hello,

I am experiencing segmentation faults with GNUstep trunk on Debian unstable
x86-64 whenever an application tries to create a NSSavePanel (according to gdb
backtrace) and would like to ask for advice on what might be causing this. It
happens with both the cairo and the art backend. I have attached a backtrace
from running and crashing Gorm, but the problem occurs with other applications
(ProjectCenter, EasyDiff) too. I hope it will be usefull.

best regards,


Niels

Relevant versions:
base,gui,back: svn r26830
cairo: 1.6.4
libart:  1.4.2





    _______________________________________________________

File Attachments:


-------------------------------------------------------
Date: Do 04 Sep 2008 10:04:10 GMT  Name: gorm-backtrace  Size: 7kB   By:
thebeing
Backtrace from running Gorm
<http://savannah.gnu.org/bugs/download.php?file_id=16430>

    _______________________________________________________

Reply to this item at:

  <http://savannah.gnu.org/bugs/?24200>

_______________________________________________
  Nachricht geschickt von/durch Savannah
  http://savannah.gnu.org/



_______________________________________________
Bug-gnustep mailing list
Bug-gnustep@...
http://lists.gnu.org/mailman/listinfo/bug-gnustep

[Paul D. Smith]

Follow-up Comment #1, bug #24200 (project gnustep):

The line [GSCurrentContext() flushGraphics]; is used in the NSSavePanel to
display progress information on big directories.
This only happends when explicitly requested by setting the property
GSSavePanelShowProgress to YES. So the simplest solution will be to set this
to NO in your defaults file.

For a general solution we will have to find out where the problem actually
happends and decide how to resolve it. I think that here a wrong context gets
found as the current context. (Although in most cases the current context will
be the application context and I see no reason, why this could be wrong)
Independent of this we should rethink, whether a flush is the right think to
do here. We are setting the window title to display the progress and when
GNUstep is displaying the window decoration itself the flush is even before
the system gets a chance to draw anything. Perhaps it would be best to remove
the whole feature? Or replace the flush by a way to get the run loop do its
job while loading huge amounts of cells?

    _______________________________________________________

Reply to this item at:

  <http://savannah.gnu.org/bugs/?24200>

_______________________________________________
  Nachricht geschickt von/durch Savannah
  http://savannah.gnu.org/



_______________________________________________
Bug-gnustep mailing list
Bug-gnustep@...
http://lists.gnu.org/mailman/listinfo/bug-gnustep

[Paul D. Smith]

Update of bug #24200 (project gnustep):

                  Status:                    None => Ready For Test        
             Assigned to:                    None => FredKiefer            
             Open/Closed:                    Open => In Test                

    _______________________________________________________

Follow-up Comment #2:

I just replaced the calls to  [GSCurrentContext() flushGraphics] with [self
flushWindow], this should work more reliable. Still when the window decoration
is drawn by GNUstep we have no garanty that the changed string will get
displayed.

    _______________________________________________________

Reply to this item at:

  <http://savannah.gnu.org/bugs/?24200>

_______________________________________________
  Nachricht geschickt von/durch Savannah
  http://savannah.gnu.org/



_______________________________________________
Bug-gnustep mailing list
Bug-gnustep@...
http://lists.gnu.org/mailman/listinfo/bug-gnustep

[Paul D. Smith]

Follow-up Comment #3, bug #24200 (project gnustep):

I just compiled and can confirm that the change fixed the issue for me, both
with GSX11HandlesWindowDecorations set to Yes and No. Besides, I can't really
see any use of setting GSSavePanelShowProgress to yes on my machine since it
neither has large enough directories nor contrained enough resources, so I
can't comment on whether the changed string gets drawn.

Cheers,


Niels

    _______________________________________________________

Reply to this item at:

  <http://savannah.gnu.org/bugs/?24200>

_______________________________________________
  Nachricht geschickt von/durch Savannah
  http://savannah.gnu.org/



_______________________________________________
Bug-gnustep mailing list
Bug-gnustep@...
http://lists.gnu.org/mailman/listinfo/bug-gnustep

[Paul D. Smith]

Update of bug #24200 (project gnustep):

                  Status:          Ready For Test => Fixed                  
             Open/Closed:                 In Test => Closed                


    _______________________________________________________

Reply to this item at:

  <http://savannah.gnu.org/bugs/?24200>

_______________________________________________
  Nachricht geschickt von/durch Savannah
  http://savannah.gnu.org/



_______________________________________________
Bug-gnustep mailing list
Bug-gnustep@...
http://lists.gnu.org/mailman/listinfo/bug-gnustep