Track binary mode, and don't enter/exit it for file transfers if it's

already in binary mode.

Should fix Sourceforge issue 91

src/syncterm/conn.c

@@ -606,10 +606,12 @@ void conn_binary_mode_on(void)
 {
 	if(conn_api.binary_mode_on)
 		conn_api.binary_mode_on();
+	conn_api.binary_mode = true;
 }
 
 void conn_binary_mode_off(void)
 {
 	if(conn_api.binary_mode_off)
 		conn_api.binary_mode_off();
+	conn_api.binary_mode = false;
 }

src/syncterm/conn.h

@@ -36,6 +36,7 @@ struct conn_api {
 	int (*close)(void);
 	void (*binary_mode_on)(void);
 	void (*binary_mode_off)(void);
+	bool binary_mode;
 	void *(*rx_parse_cb)(const void* inbuf, size_t inlen, size_t *olen);
 	void *(*tx_parse_cb)(const void* inbuf, size_t inlen, size_t *olen);
 	int log_level;

src/syncterm/term.c

@@ -1046,10 +1046,11 @@ void ascii_upload(FILE *fp)
 	fclose(fp);
 }
 
-static void transfer_complete(BOOL success)
+static void transfer_complete(BOOL success, bool was_binary)
 {
 	int timeout = success ? settings.xfer_success_keypress_timeout : settings.xfer_failure_keypress_timeout;
 
+	if (!was_binary)
 		conn_binary_mode_off();
 	if(log_fp!=NULL)
 		fflush(log_fp);
@@ -1074,6 +1075,7 @@ void zmodem_upload(struct bbslist *bbs, FILE *fp, char *path)
 	zmodem_t	zm;
 	int64_t		fsize;
 	struct zmodem_cbdata cbdata;
+	bool            was_binary = conn_api.binary_mode;
 
 	draw_transfer_window("ZMODEM Upload");
 
@@ -1081,6 +1083,7 @@ void zmodem_upload(struct bbslist *bbs, FILE *fp, char *path)
 
 	cbdata.zm=&zm;
 	cbdata.bbs=bbs;
+	if (!was_binary)
 		conn_binary_mode_on();
 	transfer_buf_len=0;
 	zmodem_init(&zm
@@ -1106,7 +1109,7 @@ void zmodem_upload(struct bbslist *bbs, FILE *fp, char *path)
 
 	fclose(fp);
 
-	transfer_complete(success);
+	transfer_complete(success, was_binary);
 }
 
 BOOL zmodem_duplicate_callback(void *cbdata, void *zm_void)
@@ -1184,6 +1187,7 @@ void zmodem_download(struct bbslist *bbs)
 	int			files_received;
 	uint64_t	bytes_received;
 	struct zmodem_cbdata cbdata;
+	bool            was_binary = conn_api.binary_mode;
 
 	if(safe_mode)
 		return;
@@ -1191,6 +1195,7 @@ void zmodem_download(struct bbslist *bbs)
 
 	zmodem_mode=ZMODEM_MODE_RECV;
 
+	if (!was_binary)
 		conn_binary_mode_on();
 	cbdata.zm=&zm;
 	cbdata.bbs=bbs;
@@ -1212,7 +1217,7 @@ void zmodem_download(struct bbslist *bbs)
 	if(files_received>1)
 		lprintf(LOG_INFO,"Received %u files (%"PRId64" bytes) successfully", files_received, bytes_received);
 
-	transfer_complete(files_received);
+	transfer_complete(files_received, was_binary);
 }
 /* End of Zmodem Stuff */
 
@@ -1423,7 +1428,9 @@ void xmodem_upload(struct bbslist *bbs, FILE *fp, char *path, long mode, int las
 	BOOL		success;
 	xmodem_t	xm;
 	int64_t		fsize;
+	bool            was_binary = conn_api.binary_mode;
 
+	if (!was_binary)
 		conn_binary_mode_on();
 
 	xmodem_init(&xm
@@ -1476,6 +1483,7 @@ void xmodem_upload(struct bbslist *bbs, FILE *fp, char *path, long mode, int las
 	}
 	else {
 		fclose(fp);
+		if (!was_binary)
 			conn_binary_mode_off();
 		return;
 	}
@@ -1499,7 +1507,7 @@ void xmodem_upload(struct bbslist *bbs, FILE *fp, char *path, long mode, int las
 
 	fclose(fp);
 
-	transfer_complete(success);
+	transfer_complete(success, was_binary);
 }
 
 BOOL xmodem_duplicate(xmodem_t *xm, struct bbslist *bbs, char *path, size_t pathsize, char *fname)
@@ -1593,6 +1601,7 @@ void xmodem_download(struct bbslist *bbs, long mode, char *path)
 	time_t	t,startfile,ftime=0;
 	int		old_hold=hold_update;
 	BOOL	extra_pass = FALSE;
+	bool            was_binary = conn_api.binary_mode;
 
 	if(safe_mode)
 		return;
@@ -1611,6 +1620,7 @@ void xmodem_download(struct bbslist *bbs, long mode, char *path)
 	else
 		return;
 
+	if (!was_binary)
 		conn_binary_mode_on();
 	xmodem_init(&xm
 		,/* cbdata */&xm
@@ -1872,7 +1882,7 @@ void xmodem_download(struct bbslist *bbs, long mode, char *path)
 end:
 	if(fp)
 		fclose(fp);
-	transfer_complete(success);
+	transfer_complete(success, was_binary);
 }
 
 /* End of X/Y-MODEM stuff */