d/freebsd-nq
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

exec: provide right hardlink name in AT_EXECPATH

Browse Source 
PR:	248184

(cherry picked from commit 0c10648fbb)
This commit is contained in:
Konstantin Belousov 2021-10-23 03:24:08 +03:00
parent b94df11d52
commit a69fb7452e
1 changed files with 28 additions and 10 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

38
sys/kern/kern_exec.c
View File

@ -401,6 +401,7 @@ do_execve(struct thread *td, struct image_args *args, struct mac *mac_p,
#endif
int error, i, orig_osrel;
uint32_t orig_fctl0;
size_t freepath_size;
static const char fexecv_proc_title[] = "(fexecv)";
imgp = &image_params;
@ -447,7 +448,8 @@ do_execve(struct thread *td, struct image_args *args, struct mac *mac_p,
*/
if (args->fname != NULL) {
NDINIT(&nd, LOOKUP, ISOPEN | LOCKLEAF | LOCKSHARED | FOLLOW |
SAVENAME | AUDITVNODE1, UIO_SYSSPACE, args->fname, td);
SAVENAME | AUDITVNODE1 | WANTPARENT,
UIO_SYSSPACE, args->fname, td);
}
SDT_PROBE1(proc, , , exec, args->fname);
@ -592,9 +594,18 @@ do_execve(struct thread *td, struct image_args *args, struct mac *mac_p,
/*
* Do the best to calculate the full path to the image file.
*/
if (args->fname != NULL && args->fname[0] == '/')
imgp->execpath = args->fname;
else {
if (args->fname != NULL) {
if (args->fname[0] == '/') {
imgp->execpath = args->fname;
} else {
VOP_UNLOCK(imgp->vp);
freepath_size = MAXPATHLEN;
if (vn_fullpath_hardlink(&nd, &imgp->execpath,
&imgp->freepath, &freepath_size) != 0)
imgp->execpath = args->fname;
vn_lock(imgp->vp, LK_SHARED | LK_RETRY);
}
} else {
VOP_UNLOCK(imgp->vp);
if (vn_fullpath(imgp->vp, &imgp->execpath,
&imgp->freepath) != 0)
@ -647,8 +658,6 @@ do_execve(struct thread *td, struct image_args *args, struct mac *mac_p,
VOP_UNSET_TEXT_CHECKED(newtextvp);
imgp->textset = false;
/* free name buffer and old vnode */
if (args->fname != NULL)
NDFREE(&nd, NDF_ONLY_PNBUF);
#ifdef MAC
mac_execve_interpreter_enter(newtextvp, &interpvplabel);
#endif
@ -657,6 +666,11 @@ do_execve(struct thread *td, struct image_args *args, struct mac *mac_p,
imgp->opened = false;
}
vput(newtextvp);
if (args->fname != NULL) {
if (nd.ni_dvp != NULL)
vrele(nd.ni_dvp);
NDFREE(&nd, NDF_ONLY_PNBUF);
}
vm_object_deallocate(imgp->object);
imgp->object = NULL;
execve_nosetid(imgp);
@ -664,9 +678,10 @@ do_execve(struct thread *td, struct image_args *args, struct mac *mac_p,
free(imgp->freepath, M_TEMP);
imgp->freepath = NULL;
/* set new name to that of the interpreter */
NDINIT(&nd, LOOKUP, ISOPEN | LOCKLEAF | LOCKSHARED | FOLLOW |
SAVENAME, UIO_SYSSPACE, imgp->interpreter_name, td);
args->fname = imgp->interpreter_name;
NDINIT(&nd, LOOKUP, ISOPEN | LOCKLEAF | LOCKSHARED | FOLLOW |
SAVENAME | WANTPARENT,
UIO_SYSSPACE, imgp->interpreter_name, td);
goto interpret;
}
@ -907,8 +922,6 @@ do_execve(struct thread *td, struct image_args *args, struct mac *mac_p,
exec_unmap_first_page(imgp);
if (imgp->vp != NULL) {
if (args->fname)
NDFREE(&nd, NDF_ONLY_PNBUF);
if (imgp->opened)
VOP_CLOSE(imgp->vp, FREAD, td->td_ucred, td);
if (imgp->textset)
@ -917,6 +930,11 @@ do_execve(struct thread *td, struct image_args *args, struct mac *mac_p,
vput(imgp->vp);
else
VOP_UNLOCK(imgp->vp);
if (args->fname != NULL) {
if (nd.ni_dvp != NULL)
vrele(nd.ni_dvp);
NDFREE(&nd, NDF_ONLY_PNBUF);
}
}
if (imgp->object != NULL)