Fix: avoid assigning to return structure on error
authorDenis Kenzior <denkenz@gmail.com>
Tue, 16 Mar 2010 20:26:53 +0000 (15:26 -0500)
committerDenis Kenzior <denkenz@gmail.com>
Tue, 16 Mar 2010 20:30:07 +0000 (15:30 -0500)
src/stkutil.c

index bdc7ae5..a5c3ac0 100644 (file)
@@ -206,12 +206,12 @@ static gboolean parse_dataobj_item(struct comprehension_tlv_iter *iter,
        if (data[0] == 0)
                return FALSE;
 
-       item->id = data[0];
        utf8 = sim_string_to_utf8(data + 1, len - 1);
 
        if (utf8 == NULL)
                return FALSE;
 
+       item->id = data[0];
        item->text = utf8;
 
        return TRUE;