Fix issue with optional PD probes.

Running (or showing info about) PDs with optional probes was not working
and causing segfaults.

The reason was a g_slist_concat() call that was applied to a list,
whereas it needs to be applied to a copy of the list.

This fixes #138.

diff --git a/sigrok-cli.c b/sigrok-cli.c
index 71c35b78d359c83e118fc8b7b72635f59d6da668..8521b9546128d17ff2c4f5fa5cae259d7d41d1a8 100644 (file)
--- a/sigrok-cli.c
+++ b/sigrok-cli.c
@@ -1014,7 +1014,7 @@ static int probes_to_gvar(struct srd_decoder *dec, GHashTable *hash,
                        (GDestroyNotify)g_variant_unref);
 
        all_probes = g_slist_copy(dec->probes);
-       all_probes = g_slist_concat(all_probes, dec->opt_probes);
+       all_probes = g_slist_concat(all_probes, g_slist_copy(dec->opt_probes));
        for (l = all_probes; l; l = l->next) {
                p = l->data;
                if (!(val_str = g_hash_table_lookup(hash, p->id)))