shithub: gefs

Info  •  Files  •  Log  •  Branches

tree: handle truncation edge case with multiple Oclearb messages

--- a/tree.c

+++ b/tree.c

@@ -511,7 +511,16 @@

 			if(m.op != Oinsert)

 				cpkvp(&v, &v, buf, sizeof(buf));

 			while(j < up->hi){

-				if(m.op == Oclearb){

+				/*

+				 * If a file is truncated multiple times,

+				 * we can end up with duplicate Oclearb

+				 * messages in the buffer.

+				 *

+				 * These Oclearb doublings will leave

+				 * v.nv with a length of 0, so we can

+				 * and should skip those

+				 */

+				if(m.op == Oclearb && v.nv != 0){

 					bp = unpackbp(v.v, v.nv);

 					freeblk(t, nil, bp);

 				}