From 7c7ed208221dbd115c7cc88faa0e42150cfa14a2 Mon Sep 17 00:00:00 2001 From: Robert Morris Date: Fri, 20 Sep 2019 13:09:26 -0400 Subject: don't leak a file system block if the buf argument to write is invalid and a usertest --- kernel/fs.c | 9 +++++++-- 1 file changed, 7 insertions(+), 2 deletions(-) (limited to 'kernel') diff --git a/kernel/fs.c b/kernel/fs.c index 5505408..53586d5 100644 --- a/kernel/fs.c +++ b/kernel/fs.c @@ -505,10 +505,15 @@ writei(struct inode *ip, int user_src, uint64 src, uint off, uint n) brelse(bp); } - if(n > 0 && off > ip->size){ - ip->size = off; + if(n > 0){ + if(off > ip->size) + ip->size = off; + // write the i-node back to disk even if the size didn't change + // because the loop above might have called bmap() and added a new + // block to ip->addrs[]. iupdate(ip); } + return n; } -- cgit v1.2.3