external
/
citus
mirror of https://github.com/citusdata/citus.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

Possible undefined behavior for bit shift in citus_stat_tenants.c

pull/7954/head
Maksim Melnikov 2025-04-07 15:09:57 +03:00
parent 8a8a493b67
commit 4fec8e2138
1 changed files with 2 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
src/backend/distributed/utils/citus_stat_tenants.c
View File

@ -50,7 +50,7 @@ ExecutorEnd_hook_type prev_ExecutorEnd = NULL;
#define ONE_QUERY_SCORE 1000000000 #define ONE_QUERY_SCORE 1000000000
#define BYTE_TO_BIT 3 /* 2^3 bits/byte */ #define BYTE_TO_BIT 3 /* 2^3 bits/byte */
#define TENANTS_STATS_FIELD_BIT_LENGTH(tenantStats, field) (sizeof((tenantStats)->field) \ #define TENANTS_STATS_FIELD_BIT_LENGTH(tenantStats, field) (sizeof((tenantStats)->field) \
<< BYTE_TO_BIT) << BYTE_TO_BIT)
static char AttributeToTenant[MAX_TENANT_ATTRIBUTE_LENGTH] = ""; static char AttributeToTenant[MAX_TENANT_ATTRIBUTE_LENGTH] = "";
static CmdType AttributeToCommandType = CMD_UNKNOWN; static CmdType AttributeToCommandType = CMD_UNKNOWN;
@ -609,6 +609,7 @@ ReduceScoreIfNecessary(TenantStats *tenantStats, TimestampTz queryTime)
if (periodCountAfterLastScoreReduction > 0) if (periodCountAfterLastScoreReduction > 0)
{ {
tenantStats->lastScoreReduction = queryTime; tenantStats->lastScoreReduction = queryTime;
/*addtional check to avoid undefined behavior */ /*addtional check to avoid undefined behavior */
tenantStats->score = (periodCountAfterLastScoreReduction < tenantStats->score = (periodCountAfterLastScoreReduction <
TENANTS_STATS_FIELD_BIT_LENGTH(tenantStats, score)) TENANTS_STATS_FIELD_BIT_LENGTH(tenantStats, score))