- check for errors building a no-such-map or no-such-domain response

  packet before sending it in response to yp_all (defect #10090)

1 files changed, 16 insertions, 6 deletions

diff --git a/src/nis.c b/src/nis.c
index 023dc3c..4629bb8 100644
--- a/src/nis.c
+++ b/src/nis.c
@@ -563,14 +563,24 @@ nis_all(struct plugin_state *state,
 	                xdr_replymsg(reply_xdrs, reply);
 			/* End of data. */
 			reply_all->more = FALSE;
-			xdr_ypresp_all(reply_xdrs, reply_all);
-			/* Queue the entire response. */
-			if (!(*reply_fragment_fn)(state, cdata,
-						  reply, reply_xdrs,
-						  FALSE, TRUE)) {
+			if (xdr_ypresp_all(reply_xdrs, reply_all)) {
+				/* Queue the entire response. */
+				if (!(*reply_fragment_fn)(state, cdata,
+							  reply, reply_xdrs,
+							  FALSE, TRUE)) {
+					slapi_log_error(SLAPI_LOG_PLUGIN,
+							state->plugin_desc->spd_id,
+							"all(%s/%s) - "
+							"error queueing "
+							"error response\n",
+							req_nokey.domain,
+							req_nokey.map);
+				}
+			} else {
 				slapi_log_error(SLAPI_LOG_PLUGIN,
 						state->plugin_desc->spd_id,
-						"all(%s/%s) - error queueing "
+						"all(%s/%s) - "
+						"error building "
 						"error response\n",
 						req_nokey.domain,
 						req_nokey.map);