Issue 8928 - all applications allows save file as to a directory, general io then crash
Summary: all applications allows save file as to a directory, general io then crash
Status: CLOSED FIXED
Alias: None
Product: General
Classification: Code
Component: code (show other issues)
Version: OOo 1.0.1
Hardware: Other All
: P1 (highest) Trivial (vote)
Target Milestone: OOo 1.0.3
Assignee: thorsten.martens
QA Contact: issues@framework
URL:
Keywords: oooqa
: 10676 (view as issue list)
Depends on:
Blocks:
 
Reported: 2002-11-03 23:35 UTC by orangepeel
Modified: 2003-03-13 11:13 UTC (History)
3 users (show)

See Also:
Issue Type: DEFECT
Latest Confirmation in: ---
Developer Difficulty: ---


Attachments
stack of 8928 (1.92 KB, text/plain)
2002-11-28 10:38 UTC, atr
no flags Details

Note You need to log in before you can comment on or make changes to this issue.
Description orangepeel 2002-11-03 23:35:33 UTC
when saving as if you type in a filename then click on a directory then the   
save button you get   
Error saving document Untitled1:   
General Error.   
General input/output error.   
   
if you now save, the application crashes  
  
tested with writer and calc presume all applications do this 
 
ui shouldn't allow this / code could save as the filename within the selected 
directory
Comment 1 prgmgr 2002-11-20 20:44:19 UTC
Thank you for using and supporting OOo.

Duplicated on RH 8.0, OOo 643:

1.  Start Writer.
2.  Enter some text.
3.  File->Save
4.  Enter text in Filename: field.
5.  Click and highlight directory.
6.  Click on "Save" button.
7.  Error dialog:

    Error saving document Untitled1:
    General Error.
    General input/output error.
8.  Do some changes.
9.  Click on Save icon in toolbar.

OOo crashes with "Aborted" text in terminal.
Comment 2 prgmgr 2002-11-20 20:45:13 UTC
Reassigning to framework team.  I hope I have the right team.
Comment 3 prgmgr 2002-11-20 20:45:35 UTC
Confirmed.
Comment 4 atr 2002-11-28 10:38:18 UTC
Created attachment 3793 [details]
stack of 8928
Comment 5 Olaf Felka 2002-11-28 11:00:56 UTC
SFX644MI! SfxObjectShell::GetState_Impl(class SfxItemSet &) + 756 bytes
SFX644MI! SfxStubSfxObjectShellGetState_Impl(class SfxShell *,class
SfxItemSet &) + 15 bytes
SFX644MI! SfxDispatcher::_FillState(class SfxSlotServer const &,class
SfxItemSet &,class SfxSlot const *) + 495 bytes
SFX644MI! SfxBindings::Update_Impl(class SfxStateCache *) + 306 bytes
SFX644MI! SfxBindings::Update(unsigned short) + 537 bytes
SFX644MI! SfxVirtualMenu::Activate(class Menu *) + 3815 bytes
SFX644MI! SfxVirtualMenu::LinkStubActivate(void *,void *) + 15 bytes
VCL644MI! Menu::Activate(void) + 76 bytes
VCL644MI! PopupMenu::ImplExecute(class Window *,class Rectangle const
&,unsigned long,class Menu *,unsigned char) + 853 bytes
VCL644MI! MenuBarWindow::ImplCreatePopup(unsigned char) + 498 bytes
VCL644MI! MenuBarWindow::ChangeHighlightItem(unsigned short,unsigned
char,unsigned char) + 932 bytes
VCL644MI! MenuBarWindow::MouseButtonDown(class MouseEvent const &) +
139 bytes
VCL644MI! ImplHandleMouseEvent(class Window *,unsigned short,unsigned
char,long,long,unsigned long,unsigned short,unsigned short) + 5483 bytes
VCL644MI! ImplWindowFrameProc(void *,class SalFrame *,unsigned
short,void const *) + 287 bytes
VCL644MI! SalFrameWndProc(struct HWND__ *,unsigned int,unsigned
int,long,int &) + 4560 bytes
VCL644MI! SalFrameWndProc(struct HWND__ *,unsigned int,unsigned
int,long,int &) + 513 bytes
VCL644MI! SalFrameWndProcW(struct HWND__ *,unsigned int,unsigned
int,long) + 38 bytes
USER32! 77e02e98()
USER32! 77e030e0()
USER32! 77e0320f()
VCL644MI! ImplSalYield(unsigned char) + 191 bytes
VCL644MI! ImplSalYield(unsigned char) + 79 bytes
VCL644MI! SalInstance::Yield(unsigned char) + 169 bytes
VCL644MI! Application::Yield(void) + 95 bytes
VCL644MI! Application::Execute(void) + 47 bytes
SOFFICE! desktop::Desktop::Main(void) + 6282 bytes
VCL644MI! SVMain(void) + 251 bytes
SOFFICE! 0112a5a8()
SOFFICE! WinMainCRTStartup + 308 bytes
KERNEL32! 77e87d08()
Comment 6 atr 2002-11-28 11:02:16 UTC
ATR->FS: I can reproduce this bug at all OS if I use our own save-as
dialog in src461- and srx644-versions.

with a non-pro I get this stack:

Comment 7 Frank Schönheit 2002-11-28 15:18:13 UTC
accepting. I don't get the crash here, but even the wrong error dialog
deserves some attention .... (I am unable to save or load or edit the
file later on)
Comment 8 Frank Schönheit 2002-11-28 15:21:39 UTC
FS->MBA: one error is that the file dialog wrongly interprets the
"save" command as finishing the dialog, though a directory is
selected, and then returns the URL for the directory as target. I'll
fix this.
However, it seems that the sfx code does not handle this error
properly, too - perhaps you may want to have a look at this, too,
though "your problem" will vanish when "my problem" is fixed.
Comment 9 Frank Schönheit 2002-11-28 15:22:36 UTC
andreas, is this really a 1.0.3 issue, or are there chances for 1.1Beta?
Comment 10 Frank Schönheit 2002-11-28 15:27:44 UTC
not yet sure how to fix this. Possible alternatives:
I when "save" is pressed in the dialog
  1. ignore any selection in the upper pane
  2. a. when a directory is selected, open this directory, as it would
        be done when no file name is entered in the respective field
     b. when a file is selected, ?
II whenever a selection is made in the upper pane, fill the file name
field with the title of the selected item. Should be no problem for
save-as, because we have a single-selection there.

I'd probably tend to II. Any opinions out there? 
Comment 11 Frank Schönheit 2002-11-28 15:31:03 UTC
btw: just found that StarOffice 5.2 did a mix of the above:
III. when selecting a directory, then do I.2.a. when selecting a file,
then do II.

sounds like an interesting solution, too :)
Comment 12 Frank Schönheit 2002-12-20 10:44:07 UTC
fixed - should be in 1.0.3
files:
svtools/source/filepicker/iodlg.cxx
  1.49.8.1 (SRC641 branch)
  1.69.6.1 (CWS os1)

Comment 13 Frank Schönheit 2003-01-08 16:34:29 UTC
re-opening for changing the ownership
Comment 14 Frank Schönheit 2003-01-08 16:36:16 UTC
FS->TM: please verify in CWS os1
Comment 15 thorsten.martens 2003-01-10 10:31:39 UTC
Problem has been fixed in the latest internal version. The behaviour
has been changed to act the same way, the windows system dialog does.
Comment 16 orangepeel 2003-01-10 10:41:13 UTC
inital reporter most happy! many thanks
Comment 17 michael.ruess 2003-01-16 10:20:44 UTC
Changed state to "verified" thus the fix can make its way into
OpenOffice 1.1 Beta build.
Comment 18 Frank Schönheit 2003-02-05 16:27:28 UTC
*** Issue 10676 has been marked as a duplicate of this issue. ***
Comment 19 michael.bemmer 2003-03-13 11:13:54 UTC
As mentioned on the qa dev list on March 5th I will close all resolved
<wontfix/duplicate/worksforme/invalid> issues. Please see this posting for details.