Skip to contentSkip to conversation

rgmanager: Fix return code when a service would deadlock

PATCH New Ryan McCabe 2 participants 2 messages Oct 13, 2012

lore
Share & tools
lore
#0 lore raw

PATCH 1/1 rgmanager: Fix return code when a service would deadlock

When we detect that starting a service would cause a deadlock, return 0 instead of -1. This fixes a crash that occurred when -1 was returned.

Resolves: rhbz#861157

---
 rgmanager/src/daemons/rg_thread.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
rgmanager/src/daemons/rg_thread.c +1 -1
index 5e551c3..b888717 100644
@@ -756,7 +756,7 @@ rt_enqueue_request(const char *resgroupname, int request,
 		logt_print(LOG_DEBUG,
 			"Failed to queue %d request for %s: Would block\n",
 			request, resgroupname);
-		return -1;
+		return 0;
 	}
 
 	ret = rq_queue_request(resgroup->rt_queue, resgroup->rt_name,
Signature · 1 line
1.7.11.7
#1 lore raw

PATCH 1/1 rgmanager: Fix return code when a service would deadlock

ACK

Quoted 24 lines · Ryan McCabe

When we detect that starting a service would cause a deadlock, return 0 instead of -1. This fixes a crash that occurred when -1 was returned.

Resolves: rhbz#861157

---
 rgmanager/src/daemons/rg_thread.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
rgmanager/src/daemons/rg_thread.c +1 -1
index 5e551c3..b888717 100644
@@ -756,7 +756,7 @@ rt_enqueue_request(const char *resgroupname, int request,
 		logt_print(LOG_DEBUG,
 			"Failed to queue %d request for %s: Would block\n",
 			request, resgroupname);
-		return -1;
+		return 0;
 	}
 
 	ret = rq_queue_request(resgroup->rt_queue, resgroup->rt_name,