X-Git-Url: https://sigrok.org/gitweb/?a=blobdiff_plain;f=session_file.c;h=e7784222f3f27c82c2428603c7aa375a6aa2e618;hb=031685005b022b98b5b1604cc6226c0cc2b959f8;hp=d6d3d8228901c93f6709c5a571c7fb65ffa05126;hpb=464d12c72aa8d83acccccbbc0bc755fbb9d542c0;p=libsigrok.git

diff --git a/session_file.c b/session_file.c
index d6d3d822..e7784222 100644
--- a/session_file.c
+++ b/session_file.c
@@ -17,20 +17,29 @@
  * along with this program.  If not, see <http://www.gnu.org/licenses/>.
  */
 
-#include "config.h"
 #include <string.h>
 #include <stdlib.h>
 #include <unistd.h>
 #include <zip.h>
 #include <glib.h>
 #include <glib/gstdio.h>
+#include "config.h"
 #include "sigrok.h"
 #include "sigrok-internal.h"
 
 extern struct sr_session *session;
 extern struct sr_device_plugin session_driver;
 
-int sr_session_load(const char *filename)
+/**
+ * Load the session from the specified filename.
+ *
+ * @param filename The name of the session file to load. Must not be NULL.
+ *
+ * @return SR_OK upon success, SR_ERR_ARG upon invalid arguments,
+ *         SR_ERR_MALLOC upon memory allocation errors, or SR_ERR upon
+ *         other errors.
+ */
+SR_API int sr_session_load(const char *filename)
 {
 	GKeyFile *kf;
 	GPtrArray *capturefiles;
@@ -43,7 +52,12 @@ int sr_session_load(const char *filename)
 	int ret, err, probenum, devcnt, i, j;
 	uint64_t tmp_u64, total_probes, enabled_probes, p;
 	char **sections, **keys, *metafile, *val, c;
-	char probename[SR_MAX_PROBENAME_LEN];
+	char probename[SR_MAX_PROBENAME_LEN + 1];
+
+	if (!filename) {
+		sr_err("session file: %s: filename was NULL", __func__);
+		return SR_ERR_ARG;
+	}
 
 	if (!(archive = zip_open(filename, 0, &err))) {
 		sr_dbg("Failed to open session file: zip error %d", err);
@@ -100,7 +114,7 @@ int sr_session_load(const char *filename)
 			for (j = 0; keys[j]; j++) {
 				val = g_key_file_get_string(kf, sections[i], keys[j], NULL);
 				if (!strcmp(keys[j], "capturefile")) {
-					device = sr_device_new(&session_driver, devcnt);
+					device = sr_dev_new(&session_driver, devcnt);
 					if (devcnt == 0)
 						/* first device, init the plugin */
 						device->plugin->init((char *)filename);
@@ -118,17 +132,17 @@ int sr_session_load(const char *filename)
 					device->plugin->set_configuration(devcnt, SR_HWCAP_CAPTURE_NUM_PROBES, &total_probes);
 					for (p = 0; p < total_probes; p++) {
 						snprintf(probename, SR_MAX_PROBENAME_LEN, "%" PRIu64, p);
-						sr_device_probe_add(device, probename);
+						sr_dev_probe_add(device, probename);
 					}
 				} else if (!strncmp(keys[j], "probe", 5)) {
 					if (!device)
 						continue;
 					enabled_probes++;
 					tmp_u64 = strtoul(keys[j]+5, NULL, 10);
-					sr_device_probe_name(device, tmp_u64, val);
+					sr_dev_probe_name(device, tmp_u64, val);
 				} else if (!strncmp(keys[j], "trigger", 7)) {
 					probenum = strtoul(keys[j]+7, NULL, 10);
-					sr_device_trigger_set(device, probenum, val);
+					sr_dev_trigger_set(device, probenum, val);
 				}
 			}
 			g_strfreev(keys);
@@ -144,6 +158,15 @@ int sr_session_load(const char *filename)
 	return SR_OK;
 }
 
+/**
+ * Save the current session to the specified file.
+ *
+ * @param filename The name of the file where to save the current session.
+ *                 Must not be NULL.
+ *
+ * @return SR_OK upon success, SR_ERR_ARG upon invalid arguments, or SR_ERR
+ *         upon other errors.
+ */
 int sr_session_save(const char *filename)
 {
 	GSList *l, *p, *d;
@@ -157,6 +180,11 @@ int sr_session_save(const char *filename)
 	uint64_t samplerate;
 	char version[1], rawname[16], metafile[32], *buf, *s;
 
+	if (!filename) {
+		sr_err("session file: %s: filename was NULL", __func__);
+		return SR_ERR_ARG;
+	}
+
 	/* Quietly delete it first, libzip wants replace ops otherwise. */
 	unlink(filename);
 	if (!(zipfile = zip_open(filename, ZIP_CREATE, &error)))
@@ -197,12 +225,12 @@ int sr_session_save(const char *filename)
 			fprintf(meta, "capturefile = logic-%d\n", devcnt);
 			fprintf(meta, "unitsize = %d\n", ds->ds_unitsize);
 			fprintf(meta, "total probes = %d\n", g_slist_length(device->probes));
-			if (sr_device_has_hwcap(device, SR_HWCAP_SAMPLERATE)) {
+			if (sr_dev_has_hwcap(device, SR_HWCAP_SAMPLERATE)) {
 				samplerate = *((uint64_t *) device->plugin->get_device_info(
 						device->plugin_index, SR_DI_CUR_SAMPLERATE));
 				s = sr_samplerate_string(samplerate);
 				fprintf(meta, "samplerate = %s\n", s);
-				free(s);
+				g_free(s);
 			}
 			probecnt = 1;
 			for (p = device->probes; p; p = p->next) {
@@ -217,8 +245,14 @@ int sr_session_save(const char *filename)
 			}
 
 			/* dump datastore into logic-n */
-			buf = malloc(ds->num_units * ds->ds_unitsize +
+			buf = g_try_malloc(ds->num_units * ds->ds_unitsize +
 				   DATASTORE_CHUNKSIZE);
+			if (!buf) {
+				sr_err("session file: %s: buf malloc failed",
+				       __func__);
+				return SR_ERR_MALLOC;
+			}
+
 			bufcnt = 0;
 			for (d = ds->chunklist; d; d = d->next) {
 				memcpy(buf + bufcnt, d->data,