bloomberg/comdb2

A comparison is always evaluated to be true

Ljiee opened this issue · 1 comments

Ljiee commented

At line 919-920 of /bdb/temptable.c,

if (cur->tbl->temp_table_type != TEMP_TABLE_TYPE_BTREE ||
        cur->tbl->temp_table_type != TEMP_TABLE_TYPE_ARRAY) {
        logmsg(LOGMSG_ERROR, "bdb_temp_table_update operation "
                             "only supported for btree or array.\n");
        return -1;
    }

this comparison is always evaluated to be true. I think the "||" should be replaced with "&&".

riverszhang89 commented

Thank you for submitting the issue.