[PATCH] xenstore: dont increment bool variable

[PATCH] xenstore: dont increment bool variable

[Juergen Gross]

Instead of incrementing a bool variable just set it to true.

Signed-off-by: Juergen Gross <jgross@suse.com>
---
 tools/xenstore/xenstored_domain.c | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/tools/xenstore/xenstored_domain.c b/tools/xenstore/xenstored_domain.c
index 4edd14eb4f..f41cf46ced 100644
--- a/tools/xenstore/xenstored_domain.c
+++ b/tools/xenstore/xenstored_domain.c
@@ -934,7 +934,8 @@ void wrl_apply_debit_actual(struct domain *domain)
 	      (long)domain->wrl_credit, (long)wrl_reserve);
 
 	if (domain->wrl_credit < 0) {
-		if (!domain->wrl_delay_logged++) {
+		if (!domain->wrl_delay_logged) {
+			domain->wrl_delay_logged = true;
 			WRL_LOG(now, "domain %ld is affected",
 				(long)domain->domid);
 		} else if (!wrl_log_last_warning) {
-- 
2.12.0


_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xen.org
https://lists.xen.org/xen-devel

[PATCH] xenstore: dont increment bool variable

[Ian Jackson]

Juergen Gross writes ("[PATCH] xenstore: dont increment bool variable"):
> Instead of incrementing a bool variable just set it to true.

Acked-by: Ian Jackson <ian.jackson@eu.citrix.com>

I will commit this to staging.  Also, I will squash it into the XSA206
fixes as I push them to the stable branches.

Ian.

_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xen.org
https://lists.xen.org/xen-devel