[fedfs-utils] [PATCH 09/13] fedfsc: Explicitly report missing NSDB paramaters

Chuck Lever chuck.lever at oracle.com
Fri Feb 1 09:21:03 PST 2013


Clean up: code in fedfs-get-limited-nsdb-params and
fedfs-get-nsdb-params should report that the server does not
have the requested connection parameters, rather than just
reporting the error code.

Signed-off-by: Chuck Lever <chuck.lever at oracle.com>
---

 src/fedfsc/fedfs-get-limited-nsdb-params.c |    5 +++++
 src/fedfsc/fedfs-get-nsdb-params.c         |    5 +++++
 2 files changed, 10 insertions(+), 0 deletions(-)

diff --git a/src/fedfsc/fedfs-get-limited-nsdb-params.c b/src/fedfsc/fedfs-get-limited-nsdb-params.c
index 7bda6eb..7bfb535 100644
--- a/src/fedfsc/fedfs-get-limited-nsdb-params.c
+++ b/src/fedfsc/fedfs-get-limited-nsdb-params.c
@@ -91,6 +91,11 @@ fedfs_get_limited_nsdb_params_print_result(FedFsGetNsdbParamsRes result)
 {
 	FedFsNsdbParams *params = &result.FedFsGetNsdbParamsRes_u.params;
 
+	if (result.status == FEDFS_ERR_NSDB_PARAMS) {
+		printf("No connection parameters found\n");
+		return;
+	}
+
 	nsdb_print_fedfsstatus(result.status);
 	if (result.status != FEDFS_OK)
 		return;
diff --git a/src/fedfsc/fedfs-get-nsdb-params.c b/src/fedfsc/fedfs-get-nsdb-params.c
index b8c7995..459c56d 100644
--- a/src/fedfsc/fedfs-get-nsdb-params.c
+++ b/src/fedfsc/fedfs-get-nsdb-params.c
@@ -95,6 +95,11 @@ fedfs_get_nsdb_params_print_result(FedFsGetNsdbParamsRes result,
 {
 	FedFsNsdbParams *params = &result.FedFsGetNsdbParamsRes_u.params;
 
+	if (result.status == FEDFS_ERR_NSDB_PARAMS) {
+		printf("No connection parameters found\n");
+		return;
+	}
+
 	nsdb_print_fedfsstatus(result.status);
 	if (result.status != FEDFS_OK)
 		return;




More information about the fedfs-utils-devel mailing list