Attachment 711845 Details for Bug 922671 – Proposed patch - part 1

[patch] Proposed patch - part 1

2013-02-05-0001-Put-handle-to-hdb-in-dispatch-on-unknown-message.patch (text/plain), 1.56 KB, created by Jan Friesse on 2013-03-18 10:00:18 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Jan Friesse

Created: 2013-03-18 10:00:18 UTC

Size: 1.56 KB

patch

obsolete

>From ffaeffdc2988fc1355eda7dd5825bd9e03838b55 Mon Sep 17 00:00:00 2001
>From: Jan Friesse <jfriesse@redhat.com>
>Date: Tue, 5 Feb 2013 11:32:07 +0100
>Subject: [PATCH 1/2] Put handle to hdb in dispatch on unknown message
>
>When we receive unknown dispatch type, we should put handle to hdb,
>because it was correctly obtained. This problem is only cfg and confdb
>libraries, cpg, evs, quorum, ... are not affected and works correctly.
>
>Signed-off-by: Jan Friesse <jfriesse@redhat.com>
>---
> lib/cfg.c    |    3 +--
> lib/confdb.c |    3 +--
> 2 files changed, 2 insertions(+), 4 deletions(-)
>
>diff --git a/lib/cfg.c b/lib/cfg.c
>index a3aee9c..af3b675 100644
>--- a/lib/cfg.c
>+++ b/lib/cfg.c
>@@ -212,7 +212,7 @@ corosync_cfg_dispatch (
> 		default:
> 			coroipcc_dispatch_put (cfg_instance->handle);
> 			error = CS_ERR_LIBRARY;
>-			goto error_nounlock;
>+			goto error_put;
> 			break;
> 		}
> 		coroipcc_dispatch_put (cfg_instance->handle);
>@@ -227,7 +227,6 @@ corosync_cfg_dispatch (
> 
> error_put:
> 	(void)hdb_handle_put (&cfg_hdb, cfg_handle);
>-error_nounlock:
> 	return (error);
> }
> 
>diff --git a/lib/confdb.c b/lib/confdb.c
>index 33e593d..7b045e3 100644
>--- a/lib/confdb.c
>+++ b/lib/confdb.c
>@@ -418,7 +418,7 @@ cs_error_t confdb_dispatch (
> 			default:
> 				coroipcc_dispatch_put (confdb_inst->handle);
> 				error = CS_ERR_LIBRARY;
>-				goto error_noput;
>+				goto error_put;
> 				break;
> 		}
> 		coroipcc_dispatch_put (confdb_inst->handle);
>@@ -433,7 +433,6 @@ cs_error_t confdb_dispatch (
> 
> error_put:
> 	(void)hdb_handle_put (&confdb_handle_t_db, handle);
>-error_noput:
> 	return (error);
> }
> 
>-- 
>1.7.1
>

Actions: View | Diff

Attachments on bug 922671: 711845 | 711846 | 711848 | 711850 | 711875 | 711877