From 331abd32b93b6f86a62b871592394f86349f9031 Mon Sep 17 00:00:00 2001 From: anserper Date: Wed, 13 Feb 2008 21:23:34 +0000 Subject: [PATCH] Branch HEAD b=13285 i=johann i=deen MIME-Version: 1.0 Content-Type: text/plain; charset=utf8 Content-Transfer-Encoding: 8bit �� fixes some return code issues (refer to https://bugzilla.lustre.org/attachment.cgi?id=15278&action=edit) --- lustre/obdclass/lprocfs_status.c | 8 ++++++-- 1 file changed, 6 insertions(+), 2 deletions(-) diff --git a/lustre/obdclass/lprocfs_status.c b/lustre/obdclass/lprocfs_status.c index a4e61bf..4a65259 100644 --- a/lustre/obdclass/lprocfs_status.c +++ b/lustre/obdclass/lprocfs_status.c @@ -1507,7 +1507,9 @@ int lprocfs_obd_rd_recovery_status(char *page, char **start, off_t off, goto out; if (obd->obd_max_recoverable_clients == 0) { - lprocfs_obd_snprintf(&page, size, &len, "INACTIVE\n"); + if (lprocfs_obd_snprintf(&page, size, &len, "INACTIVE\n") <= 0) + goto out; + goto fclose; } @@ -1572,7 +1574,9 @@ int lprocfs_obd_rd_recovery_status(char *page, char **start, off_t off, obd->obd_requests_queued_for_recovery) <= 0) goto out; - lprocfs_obd_snprintf(&page, size, &len, "next_transno: "LPD64"\n", obd->obd_next_recovery_transno); + if (lprocfs_obd_snprintf(&page, size, &len, "next_transno: "LPD64"\n", + obd->obd_next_recovery_transno) <= 0) + goto out; fclose: *eof = 1; -- 1.8.3.1