<div dir="ltr">Hey Aleksander, <div><br></div><div>Yep, that fixes it in the same way but clearer! Don't need to pass in *out (or pass in 0).</div><div>Didn't check if the other bits were for anything so left them alone in my patch.</div><div><br></div><div>Thanks,</div><div><br></div><div>--</div><div>Paul</div></div><div class="gmail_extra"><br><div class="gmail_quote">On 15 February 2017 at 16:47, Aleksander Morgado <span dir="ltr"><<a href="mailto:aleksander@aleksander.es" target="_blank">aleksander@aleksander.es</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">We OR each flag value found in the output directly, so make sure that<br>
output is clear before adding any new flag.<br>
<br>
Reported-by: Paul Gildea <<a href="mailto:gildeap@tcd.ie">gildeap@tcd.ie</a>><br>
---<br>
<br>
Hey Paul,<br>
<br>
Could you try this patch and see if you still have issues?<br>
<br>
Also fixed the same issue in the device open flags, btw.<br>
<br>
---<br>
 src/qmicli/qmicli-helpers.c | 4 ++++<br>
 1 file changed, 4 insertions(+)<br>
<br>
diff --git a/src/qmicli/qmicli-helpers.c b/src/qmicli/qmicli-helpers.c<br>
index 2a2dad2..46e17db 100644<br>
--- a/src/qmicli/qmicli-helpers.c<br>
+++ b/src/qmicli/qmicli-helpers.c<br>
@@ -237,6 +237,8 @@ qmicli_read_rat_mode_pref_<wbr>from_string (const gchar *str,<br>
     type = qmi_nas_rat_mode_preference_<wbr>get_type ();<br>
     flags_class = G_FLAGS_CLASS (g_type_class_ref (type));<br>
<br>
+    *out = 0;<br>
+<br>
     items = g_strsplit_set (str, "|", 0);<br>
     for (iter = items; iter && *iter && success; iter++) {<br>
         if (!*iter[0])<br>
@@ -451,6 +453,8 @@ qmicli_read_net_open_flags_<wbr>from_string (const gchar *str,<br>
     type = qmi_device_open_flags_get_type ();<br>
     flags_class = G_FLAGS_CLASS (g_type_class_ref (type));<br>
<br>
+    *out = 0;<br>
+<br>
     items = g_strsplit_set (str, "|", 0);<br>
     for (iter = items; iter && *iter && success; iter++) {<br>
         if (!*iter[0])<br>
--<br>
2.11.1<br>
</blockquote></div><br></div>