From: Troy Crosley <troycrosley@gmail.com>
Date: Tue, 1 Sep 2020 17:27:39 +0000 (-0400)
Subject: Avoid unnecessary assertion.
X-Git-Url: http://xenbits.xensource.com/gitweb?a=commitdiff_plain;h=dd5c2a19fdaced855a92e940cb23915790398df6;p=pvdrivers%2Fwin%2Fxeniface.git

Avoid unnecessary assertion.

Add the missing brackets to an if statement so that FdoS4ToS3() is not
always called in __FdoSetSystemPowerUp().

Signed-off-by: Troy Crosley <troycrosley@gmail.com>
Reviewed-by: Owen Smith <owen.smith@citrix.com>
---

diff --git a/src/xeniface/fdo.c b/src/xeniface/fdo.c
index 1f859cd..6aabf96 100644
--- a/src/xeniface/fdo.c
+++ b/src/xeniface/fdo.c
@@ -1740,9 +1740,10 @@ __FdoSetSystemPowerUp(
         goto done;
 
     if (SystemState < PowerSystemHibernate &&
-        __FdoGetSystemPowerState(Fdo) >= PowerSystemHibernate)
+        __FdoGetSystemPowerState(Fdo) >= PowerSystemHibernate) {
         __FdoSetSystemPowerState(Fdo, PowerSystemHibernate);
         FdoS4ToS3(Fdo);
+    }
 
     Info("%s -> %s\n",
          PowerSystemStateName(__FdoGetSystemPowerState(Fdo)),