That was the wrong fix. MSG_ERROR should set the error and the client should

use the error and exit on MSG_EXIT (it was being handled in the default
case). Undo the last change, move the errstr check into the MSG_EXIT case, and
add a comment.

1 files changed, 6 insertions, 5 deletions

diff --git a/client.c b/client.c
index bd062c69..8724c35e 100644
--- a/client.c
+++ b/client.c
@@ -214,16 +214,17 @@ client_main(struct client_ctx *cctx)
 		printf("[server exited]\n");
 		return (0);
 	case CCTX_EXIT:
+		if (cctx->errstr != NULL) {
+			printf("[error: %s]\n", cctx->errstr);
+			return (1);
+		}
 		printf("[exited]\n");
 		return (0);
 	case CCTX_DETACH:
 		printf("[detached]\n");
 		return (0);
-	case CCTX_ERROR:
-		printf("[error: %s]\n", cctx->errstr);
-		return (1);
 	default:
-		printf("[error: unknown error]\n");
+		printf("[unknown error]\n");
 		return (1);
 	}
 }
@@ -277,8 +278,8 @@ client_msg_dispatch(struct client_ctx *cctx)
 			memcpy(&printdata, imsg.data, sizeof printdata);
 
 			printdata.msg[(sizeof printdata.msg) - 1] = '\0';
+			/* Error string used after exit message from server. */
 			cctx->errstr = xstrdup(printdata.msg);
-			cctx->exittype = CCTX_ERROR;
 			imsg_free(&imsg);
 			return (-1);
 		case MSG_EXIT: