Do not read past the end of a text buffer looking for a zero terminator, as

that space might not be initialized.  If the buffer is owned, just set the
null terminator.  This is a better fix for the OSSFuzz-detected
use-of-initialized-value problem.

FossilOrigin-Name: 931bccb0cc290b8bf3027641e7a7fac30e3244d7dc84aa9e38b24b7e9544ca06

1 files changed, 1 insertions, 1 deletions

diff --git a/src/vdbemem.c b/src/vdbemem.c
index 3f845452c..b5a794ae8 100644
--- a/src/vdbemem.c
+++ b/src/vdbemem.c
@@ -328,8 +328,8 @@ void sqlite3VdbeMemZeroTerminateIfAble(Mem *pMem){
   if( pMem->flags & MEM_Dyn ){
     if( pMem->xDel==sqlite3_free
      && sqlite3_msize(pMem->z) >= (u64)(pMem->n+1)
-     && pMem->z[pMem->n]==0 
     ){
+      pMem->z[pMem->n] = 0;
       pMem->flags |= MEM_Term;
       return;
     }