From 766b752c18e83d1ef91e7b44b433023ed5fb7f0c Mon Sep 17 00:00:00 2001 From: dmchen Date: Thu, 4 May 2023 18:35:37 +0800 Subject: [PATCH] if statement for overflow --- source/dnode/mnode/impl/src/mndStb.c | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/source/dnode/mnode/impl/src/mndStb.c b/source/dnode/mnode/impl/src/mndStb.c index 939080be47..3407d4a1dc 100644 --- a/source/dnode/mnode/impl/src/mndStb.c +++ b/source/dnode/mnode/impl/src/mndStb.c @@ -797,7 +797,7 @@ int32_t mndBuildStbFromReq(SMnode *pMnode, SStbObj *pDst, SMCreateStbReq *pCreat return -1; } - if(pDst->nextColId > 0 && pDst->nextColId < 0x7fff - pDst->numOfColumns - pDst->numOfTags){ + if(pDst->nextColId < 0 || pDst->nextColId >= 0x7fff - pDst->numOfColumns - pDst->numOfTags){ terrno = TSDB_CODE_MND_BIG_FIELD_VALUE; return -1; } @@ -932,7 +932,7 @@ static int32_t mndBuildStbFromAlter(SStbObj *pStb, SStbObj *pDst, SMCreateStbReq return -1; } - if(pDst->nextColId > 0 && pDst->nextColId < 0x7fff - pDst->numOfColumns - pDst->numOfTags){ + if(pDst->nextColId < 0 && pDst->nextColId >= 0x7fff - pDst->numOfColumns - pDst->numOfTags){ terrno = TSDB_CODE_MND_BIG_FIELD_VALUE; return -1; } @@ -1164,7 +1164,7 @@ static int32_t mndAddSuperTableTag(const SStbObj *pOld, SStbObj *pNew, SArray *p return -1; } - if(pNew->nextColId > 0 && pNew->nextColId < 0x7fff - ntags){ + if(pNew->nextColId < 0 && pNew->nextColId >= 0x7fff - ntags){ terrno = TSDB_CODE_MND_BIG_FIELD_VALUE; return -1; } @@ -1476,7 +1476,7 @@ static int32_t mndAddSuperTableColumn(const SStbObj *pOld, SStbObj *pNew, SArray return -1; } - if(pNew->nextColId > 0 && pNew->nextColId < 0x7fff - ncols){ + if(pNew->nextColId < 0 && pNew->nextColId >= 0x7fff - ncols){ terrno = TSDB_CODE_MND_BIG_FIELD_VALUE; return -1; }