Xconfig::xfree3::set_resolution and Xconfig::xfree4::set_resolution must not modify parameter $Screen before calling Xconfig::xfreeX::set_resolution (otherwise the resolution setting is done on only one device which is bad for multi-head)

1 files changed, 4 insertions, 5 deletions

diff --git a/perl-install/Xconfig/xfree3.pm b/perl-install/Xconfig/xfree3.pm
index 9235f8f96..9b69b3cb6 100644
--- a/perl-install/Xconfig/xfree3.pm
+++ b/perl-install/Xconfig/xfree3.pm
@@ -86,13 +86,12 @@ sub set_wacoms {
 sub depths { 8, 15, 16, 24, 32 }
 sub set_resolution {
     my ($raw_X, $resolution, $Screen) = @_;
-    $Screen ||= $raw_X->get_default_screen or return {};
 
     $resolution = +{ %$resolution };
-
-    #- use framebuffer if Screen is
-    $resolution->{fbdev} = 1 if val($Screen->{Driver}) eq 'fbdev';
-
+    if (my $Screen_ = $Screen || $raw_X->get_default_screen) {
+	#- use framebuffer if Screen is
+	$resolution->{fbdev} = 1 if val($Screen_->{Driver}) eq 'fbdev';
+    }
     $raw_X->SUPER::set_resolution($resolution, $Screen);
 }