diff options
author | Christophe Fergeau <cfergeau@redhat.com> | 2014-09-23 17:23:45 +0200 |
---|---|---|
committer | Christophe Fergeau <cfergeau@redhat.com> | 2015-04-01 10:58:43 +0200 |
commit | 8003f9e01f8fc6dc73de2ec2af303af556e96e84 (patch) | |
tree | 3648b2b7136df4b0eebada08ce8e62cd52c1bb68 | |
parent | 7329ad767f5413459cc98210eea754a5de95af1e (diff) | |
download | vd_agent-8003f9e01f8fc6dc73de2ec2af303af556e96e84.tar.gz vd_agent-8003f9e01f8fc6dc73de2ec2af303af556e96e84.tar.xz vd_agent-8003f9e01f8fc6dc73de2ec2af303af556e96e84.zip |
Remove unneeded NULL check
Since previous commit, vdagent_x11_set_monitor_config() will return
early if 'curr' is NULL so we don't need to explicitly check its value.
Issue reported by the Coverity checker.
-rw-r--r-- | src/vdagent-x11-randr.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/vdagent-x11-randr.c b/src/vdagent-x11-randr.c index f48f045..c332a40 100644 --- a/src/vdagent-x11-randr.c +++ b/src/vdagent-x11-randr.c @@ -795,7 +795,7 @@ void vdagent_x11_set_monitor_config(struct vdagent_x11 *x11, width_mm, height_mm); if (vdagent_x11_restore_error_handler(x11)) { syslog(LOG_ERR, "XRRSetScreenSize failed, not enough mem?"); - if (!fallback && curr) { + if (!fallback) { syslog(LOG_WARNING, "Restoring previous config"); vdagent_x11_set_monitor_config(x11, curr, 1); free(curr); |