Changeset: 8a1c5cb85934 for MonetDB
URL: http://dev.monetdb.org/hg/MonetDB?cmd=changeset;node=8a1c5cb85934
Modified Files:
        sql/storage/bat/bat_table.c
Branch: default
Log Message:

Don't call va_arg out of bounds because this triggers the address sanitizer.


diffs (15 lines):

diff --git a/sql/storage/bat/bat_table.c b/sql/storage/bat/bat_table.c
--- a/sql/storage/bat/bat_table.c
+++ b/sql/storage/bat/bat_table.c
@@ -241,8 +241,10 @@ table_insert(sql_trans *tr, sql_table *t
        int cnt = 0;
 
        va_start(va, t);
-       for (val = va_arg(va, void *); n && val; n = n->next, val = va_arg(va, 
void *))
+       for (; n; n = n->next)
        {
+               val = va_arg(va, void *);
+               if (!val) break;
                sql_column *c = n->data;
                store_funcs.append_col(tr, c, val, c->type.type->localtype);
                cnt++;
_______________________________________________
checkin-list mailing list
checkin-list@monetdb.org
https://www.monetdb.org/mailman/listinfo/checkin-list

Reply via email to