yuheho7749 / uwufs

GNU General Public License v3.0
1 stars 1 forks source link

append_dblk operation can storage leak blocks #33

Open yuheho7749 opened 3 hours ago

yuheho7749 commented 3 hours ago

If malloc block fails (due to no storage). It will set the inode's indirect pointer to 0, but still returns the original block no, meaning it reported the operation as successful when it actually failed. This also storage leaks blocks allocated in the process.

yuheho7749 commented 3 hours ago

Fix it after I merge the dev/indirect-blocks branch. I have other fixes/checks in place that makes this easier to fix. Namely, the tracking of how many free blocks left.