Bug 224944

Summary:	Yast2-vm: Interface freezes after aborting install source setup
Product:	[openSUSE] openSUSE 10.2	Reporter:	Lynn Bendixsen <lbendixs>
Component:	YaST2	Assignee:	Ladislav Slezák <lslezak>
Status:	RESOLVED FIXED	QA Contact:	Jason Douglas <jdouglas>
Severity:	Critical
Priority:	P5 - None	CC:	carnold, jdouglas
Version:	RC 4
Target Milestone:	---
Hardware:	Other
OS:	Other
Whiteboard:
Found By:	Other	Services Priority:
Business Priority:		Blocker:	---
Marketing QA Status:	---	IT Deployment:	---
Attachments:	yast logs

Description Lynn Bendixsen 2006-11-30 16:29:19 UTC

Aborting after starting to add an install source in yast2-vm causes freeze of yast2-vm interface.

To duplicate:
1: start yast2-vm on oS 10.2 (probably occurs on SLES 10 too)
2. select add -> "run an ..." -> Operating system installation" -> "Network Install Source" -> "Configure"
3. Select "back" then "abort" then "Yes" to abort catalog configuration.
4. On the yast2-vm setup screen all options are grayed out and nothing is selectable.
5. Shutting down yast2-vm yeilds an Error Window:  "No Dialog existing during UserInput(). Please check the log file!"

Log files attached.

Comment 1 Lynn Bendixsen 2006-11-30 16:32:26 UTC

Created attachment 107676 [details]
yast logs

Comment 2 Lynn Bendixsen 2006-11-30 16:34:04 UTC

This is more serious than first realized.

Adding an install source causes a similar problem and brings down yast2-vm.

Increasing severity.

Comment 3 Charles Arnold 2006-12-01 23:30:30 UTC

This problem has shown up in 10.2 because of a change in packager/src/client/inst_source.ycp.  An additional UI::CloseDialog was added at the bottom of the file to handle a problem with SLP (#219759).  This actually generates a mismatched set of open/closes and the extra close causes our dialog to get closed.  By removing the UI::CloseDialog from SummaryDialog(), the problem is taken care of.

--- src/clients/inst_source.ycp.orig    2006-12-01 16:20:33.000000000 -0700
+++ src/clients/inst_source.ycp 2006-12-01 16:20:40.000000000 -0700
@@ -562,8 +562,6 @@

     } until ( exit );

-    // UI::CloseDialog();
-
     y2debug( "Return: %1", input );

     return input;

Comment 4 Lukas Ocilka 2006-12-02 20:22:14 UTC

Lslezak: it seems there is problem with some Open/Close_Dialog calls when adding an installation source, they do not match.

Comment 5 Ladislav Slezák 2006-12-06 14:35:55 UTC

Charles, thanks for the patch!

Fixed in yast2-packager-2.15.1