From 1b22da00390f69fcdfd57135e10f141750e47f00 Mon Sep 17 00:00:00 2001 From: Matthias Clasen Date: Mon, 20 Jul 2015 17:33:28 -0400 Subject: [PATCH 1/2] Monitor-config: Fix a copy-paste error The code was checking width twice, instead of width and height, as was clearly the intention. Coverity pointed this out. https://bugzilla.gnome.org/show_bug.cgi?id=752551 --- src/backends/meta-monitor-config.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/src/backends/meta-monitor-config.c b/src/backends/meta-monitor-config.c index eae142a..ba0ae57 100644 --- a/src/backends/meta-monitor-config.c +++ b/src/backends/meta-monitor-config.c @@ -489,8 +489,8 @@ handle_end_element (GMarkupParseContext *context, } else { - if (parser->output.rect.width == 0 && - parser->output.rect.width == 0) + if (parser->output.rect.width == 0 || + parser->output.rect.height == 0) parser->output.enabled = FALSE; else parser->output.enabled = TRUE; -- 2.4.5