procyberian/emacs
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Avoid dereference of a freed vnode's operations table

Browse source 
* src/androidvfs.c (android_renameat_noreplace):
(android_rename): Free vdst using vdst->ops, not vp->ops.
This commit is contained in:
Po Lu 2023-07-27 18:17:12 +08:00
parent 24711be050
commit c85222a593
1 changed files with 2 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

4
src/androidvfs.c
View file

@ -5581,7 +5581,7 @@ android_renameat_noreplace (int srcfd, const char *src,
/* Now try to rename vp to vdst. */
rc = (*vp->ops->rename) (vp, vdst, true);
(*vp->ops->close) (vp);
(*vp->ops->close) (vdst);
(*vdst->ops->close) (vdst);
return rc;
error1:
@ -5613,7 +5613,7 @@ android_rename (const char *src, const char *dst)
/* Now try to rename vp to vdst. */
rc = (*vp->ops->rename) (vp, vdst, false);
(*vp->ops->close) (vp);
(*vp->ops->close) (vdst);
(*vdst->ops->close) (vdst);
return rc;
error1: