From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 18656 invoked by alias); 15 Jul 2008 14:56:44 -0000 Received: (qmail 18573 invoked by uid 9452); 15 Jul 2008 14:56:42 -0000 Date: Tue, 15 Jul 2008 14:56:00 -0000 Message-ID: <20080715145642.18557.qmail@sourceware.org> From: ccaulfield@sourceware.org To: cluster-cvs@sources.redhat.com, cluster-devel@redhat.com Subject: Cluster Project branch, master, updated. cluster-2.99.06-1-g6a9d68c X-Git-Refname: refs/heads/master X-Git-Reftype: branch X-Git-Oldrev: 12e98c1895efea3c486bfe7aeaf57aca334fc05f X-Git-Newrev: 6a9d68c3ecfaaf5795996c3db89969c543f22025 Mailing-List: contact cluster-cvs-help@sourceware.org; run by ezmlm Precedence: bulk List-Id: List-Subscribe: List-Post: List-Help: , Sender: cluster-cvs-owner@sourceware.org X-SW-Source: 2008-q3/txt/msg00064.txt.bz2 This is an automated email from the git hooks/post-receive script. It was generated because a ref change was pushed to the repository containing the project "Cluster Project". http://sources.redhat.com/git/gitweb.cgi?p=cluster.git;a=commitdiff;h=6a9d68c3ecfaaf5795996c3db89969c543f22025 The branch, master has been updated via 6a9d68c3ecfaaf5795996c3db89969c543f22025 (commit) from 12e98c1895efea3c486bfe7aeaf57aca334fc05f (commit) Those revisions listed above that are new to this repository have not appeared on any other notification email; so we list those revisions in full, below. - Log ----------------------------------------------------------------- commit 6a9d68c3ecfaaf5795996c3db89969c543f22025 Author: Christine Caulfield Date: Tue Jul 15 15:55:13 2008 +0100 [CCS] Set errno when an error occurs. Set errno to the error code to be consistent with other libraries. Signed-off-by: Christine Caulfield ----------------------------------------------------------------------- Summary of changes: config/libs/libccsconfdb/libccs.c | 42 +++++++++++++++++++++++++++++-------- 1 files changed, 33 insertions(+), 9 deletions(-) diff --git a/config/libs/libccsconfdb/libccs.c b/config/libs/libccsconfdb/libccs.c index 278c9c7..bf9216d 100644 --- a/config/libs/libccsconfdb/libccs.c +++ b/config/libs/libccsconfdb/libccs.c @@ -70,8 +70,10 @@ static int dump_objdb_buff(confdb_handle_t dump_handle, unsigned int parent_obje int res; res = confdb_key_iter_start(dump_handle, parent_object_handle); - if (res != SA_AIS_OK) + if (res != SA_AIS_OK) { + errno = ENOMEM; return -1; + } if (!*buffer || ((*buffer) && !strlen(*buffer))) { snprintf(temp, PATH_MAX - 1, "\n\n"); @@ -92,15 +94,19 @@ static int dump_objdb_buff(confdb_handle_t dump_handle, unsigned int parent_obje } res = confdb_object_iter_start(dump_handle, parent_object_handle); - if (res != SA_AIS_OK) + if (res != SA_AIS_OK) { + errno = ENOMEM; return -1; + } while ( (res = confdb_object_iter(dump_handle, parent_object_handle, &object_handle, object_name, &object_name_len)) == SA_AIS_OK) { unsigned int parent; res = confdb_object_parent_get(dump_handle, object_handle, &parent); - if (res != SA_AIS_OK) + if (res != SA_AIS_OK) { + errno = EINVAL; return -1; + } object_name[object_name_len] = '\0'; @@ -113,8 +119,10 @@ static int dump_objdb_buff(confdb_handle_t dump_handle, unsigned int parent_obje add_to_buffer(temp, buffer, size); res = dump_objdb_buff(dump_handle, object_handle, buffer, size); - if(res) + if(res) { + errno = -res; return res; + } if (object_handle != parent_object_handle) { snprintf(temp, PATH_MAX - 1, "\n", object_name); @@ -176,14 +184,17 @@ int ccs_connect(void){ return 1; res = confdb_initialize (&handle, &callbacks); - if (res != SA_AIS_OK) + if (res != SA_AIS_OK) { + errno = ENOMEM; return -1; + } if(!fullxpath) xpathlite_init(); else if (xpathfull_init() < 0) { ccs_disconnect(1); + errno = ENOMEM; return -1; } @@ -206,6 +217,7 @@ int ccs_force_connect(const char *cluster_name, int blocking){ if (res < 0) sleep(1); } + errno = -res; return res; } else return ccs_connect(); @@ -241,8 +253,10 @@ int ccs_disconnect(int desc){ } res = confdb_finalize (handle); - if (res != CONFDB_OK) + if (res != CONFDB_OK) { + errno = EINVAL; return -1; + } handle = 0; @@ -260,15 +274,19 @@ static int tokenizer() { while (curpos <= end) { index++; - if (strncmp(curpos, "/", 1)) + if (strncmp(curpos, "/", 1)) { + errno = EINVAL; return -1; + } memset(curpos, 0, 1); curpos = curpos + 1; next = strstr(curpos, "/"); - if (next == curpos) + if (next == curpos) { + errno = EINVAL; return -1; + } if(!next) return index; @@ -279,6 +297,7 @@ static int tokenizer() { curpos = strstr(strstr(curpos, "]"), "/"); } + errno = EINVAL; return -1; } @@ -400,6 +419,7 @@ static int path_dive(int tokens) return 0; fail: + errno = EINVAL; return -1; } @@ -504,6 +524,7 @@ static int get_data(char **rtn, char *curpos, int list, int is_oldlist) return 0; fail: + errno = EINVAL; return -1; } @@ -567,6 +588,7 @@ int _ccs_get_xpathlite(int desc, const char *query, char **rtn, int list) strncpy(previous_query, query, PATH_MAX-1); fail: + errno = -res; return res; } @@ -666,6 +688,7 @@ fail: if(obj) xmlXPathFreeObject(obj); + errno = -res; return res; } @@ -697,7 +720,8 @@ int ccs_get_list(int desc, const char *query, char **rtn){ * Returns: 0 on success, < 0 on failure */ int ccs_set(int desc, const char *path, char *val){ - return -ENOSYS; + errno = ENOSYS; + return -1; } /** hooks/post-receive -- Cluster Project