X-Git-Url: https://sigrok.org/gitweb/?p=sigrok-cli.git;a=blobdiff_plain;f=input.c;h=cce0d9ff916b9a159f0f3400d80bd5d96df263eb;hp=0c8d48670ac8f17ea5e9e666c75a1dfe4160dfbf;hb=HEAD;hpb=cfad6a304754649dbd8841af8a15ddba16a69cc6

diff --git a/input.c b/input.c
index 0c8d486..cce0d9f 100644
--- a/input.c
+++ b/input.c
@@ -44,6 +44,7 @@ static void load_input_file_module(struct df_arg_desc *df_arg)
 	ssize_t len;
 	char *mod_id;
 	gboolean is_stdin;
+	gboolean push_scan_data;
 
 	if (!sr_input_list())
 		g_critical("No supported input formats available.");
@@ -51,11 +52,12 @@ static void load_input_file_module(struct df_arg_desc *df_arg)
 	mod_id = NULL;
 	mod_args = NULL;
 	if (opt_input_format) {
-		mod_args = parse_generic_arg(opt_input_format, TRUE);
+		mod_args = parse_generic_arg(opt_input_format, TRUE, NULL);
 		mod_id = g_hash_table_lookup(mod_args, "sigrok_key");
 	}
 
 	is_stdin = strcmp(opt_input_file, "-") == 0;
+	push_scan_data = FALSE;
 	fd = 0;
 	buf = g_string_sized_new(CHUNK_SIZE);
 	if (mod_id) {
@@ -66,7 +68,7 @@ static void load_input_file_module(struct df_arg_desc *df_arg)
 		if ((options = sr_input_options_get(imod))) {
 			mod_opts = generic_arg_to_opt(options, mod_args);
 			(void)warn_unknown_keys(options, mod_args, NULL);
-			sr_output_options_free(options);
+			sr_input_options_free(options);
 		} else {
 			mod_opts = NULL;
 		}
@@ -103,11 +105,12 @@ static void load_input_file_module(struct df_arg_desc *df_arg)
 					g_critical("Failed to load %s: %s.", opt_input_file,
 							g_strerror(errno));
 			}
-			if ((len = read(fd, buf->str, CHUNK_SIZE)) < 1)
+			if ((len = read(fd, buf->str, buf->allocated_len)) < 1)
 				g_critical("Failed to read %s: %s.", opt_input_file,
 						g_strerror(errno));
 			buf->len = len;
 			sr_input_scan_buffer(buf, &in);
+			push_scan_data = TRUE;
 		}
 		if (!in)
 			g_critical("Error: no input module found for this file.");
@@ -116,24 +119,46 @@ static void load_input_file_module(struct df_arg_desc *df_arg)
 	df_arg->session = session;
 	sr_session_datafeed_callback_add(session, datafeed_in, df_arg);
 
+	/*
+	 * Implementation detail: The combination of reading from stdin
+	 * and automatic file format detection may have pushed the first
+	 * chunk of input data into the input module's data accumulator,
+	 * _bypassing_ the .receive() callback. It is essential to call
+	 * .receive() before calling .end() for files of size smaller than
+	 * CHUNK_SIZE (which is a typical case). So that sdi becomes ready.
+	 * Fortunately all input modules accept .receive() calls with
+	 * a zero length, and inspect whatever was accumulated so far.
+	 *
+	 * After that optional initial push of data which was queued
+	 * above during format detection, continue reading remaining
+	 * chunks from the input file until EOF is seen.
+	 */
 	got_sdi = FALSE;
 	while (TRUE) {
 		g_string_truncate(buf, 0);
-		len = read(fd, buf->str, CHUNK_SIZE);
+		if (push_scan_data)
+			len = 0;
+		else
+			len = read(fd, buf->str, buf->allocated_len);
 		if (len < 0)
 			g_critical("Read failed: %s", g_strerror(errno));
-		if (len == 0)
+		if (len == 0 && !push_scan_data)
 			/* End of file or stream. */
 			break;
+		push_scan_data = FALSE;
 		buf->len = len;
-		if (sr_input_send(in, buf) != SR_OK)
+		if (sr_input_send(in, buf) != SR_OK) {
+			g_critical("File import failed (read)");
 			break;
+		}
 
 		sdi = sr_input_dev_inst_get(in);
 		if (!got_sdi && sdi) {
 			/* First time we got a valid sdi. */
-			if (select_channels(sdi) != SR_OK)
+			if (select_channels(sdi) != SR_OK) {
+				g_critical("File import failed (channels)");
 				return;
+			}
 			if (sr_session_dev_add(session, sdi) != SR_OK) {
 				g_critical("Failed to use device.");
 				sr_session_destroy(session);
@@ -145,10 +170,10 @@ static void load_input_file_module(struct df_arg_desc *df_arg)
 	sr_input_end(in);
 	sr_input_free(in);
 	g_string_free(buf, TRUE);
+	close(fd);
 
 	df_arg->session = NULL;
 	sr_session_destroy(session);
-
 }
 
 void load_input_file(gboolean do_props)