[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Qemu-devel] [PATCH 05/14] linuxload: id_change was a write only variabl
From: |
Juan Quintela |
Subject: |
[Qemu-devel] [PATCH 05/14] linuxload: id_change was a write only variable |
Date: |
Thu, 2 Jun 2011 13:53:40 +0200 |
Signed-off-by: Juan Quintela <address@hidden>
---
linux-user/linuxload.c | 25 +------------------------
1 files changed, 1 insertions(+), 24 deletions(-)
diff --git a/linux-user/linuxload.c b/linux-user/linuxload.c
index ac8c486..62ebc7e 100644
--- a/linux-user/linuxload.c
+++ b/linux-user/linuxload.c
@@ -26,22 +26,6 @@ abi_long memcpy_to_target(abi_ulong dest, const void *src,
return 0;
}
-static int in_group_p(gid_t g)
-{
- /* return TRUE if we're in the specified group, FALSE otherwise */
- int ngroup;
- int i;
- gid_t grouplist[NGROUPS];
-
- ngroup = getgroups(NGROUPS, grouplist);
- for(i = 0; i < ngroup; i++) {
- if(grouplist[i] == g) {
- return 1;
- }
- }
- return 0;
-}
-
static int count(char ** vec)
{
int i;
@@ -57,7 +41,7 @@ static int prepare_binprm(struct linux_binprm *bprm)
{
struct stat st;
int mode;
- int retval, id_change;
+ int retval;
if(fstat(bprm->fd, &st) < 0) {
return(-errno);
@@ -73,14 +57,10 @@ static int prepare_binprm(struct linux_binprm *bprm)
bprm->e_uid = geteuid();
bprm->e_gid = getegid();
- id_change = 0;
/* Set-uid? */
if(mode & S_ISUID) {
bprm->e_uid = st.st_uid;
- if(bprm->e_uid != geteuid()) {
- id_change = 1;
- }
}
/* Set-gid? */
@@ -91,9 +71,6 @@ static int prepare_binprm(struct linux_binprm *bprm)
*/
if ((mode & (S_ISGID | S_IXGRP)) == (S_ISGID | S_IXGRP)) {
bprm->e_gid = st.st_gid;
- if (!in_group_p(bprm->e_gid)) {
- id_change = 1;
- }
}
retval = read(bprm->fd, bprm->buf, BPRM_BUF_SIZE);
--
1.7.5.2
- [Qemu-devel] [PATCH 00/14] More gcc 4.6 warnings fixes, Juan Quintela, 2011/06/02
- [Qemu-devel] [PATCH 01/14] kvm: remove fop write only variable, Juan Quintela, 2011/06/02
- [Qemu-devel] [PATCH 02/14] tcg: define and set call_type only when it is used, Juan Quintela, 2011/06/02
- [Qemu-devel] [PATCH 04/14] xen: pentry is not used in this function, Juan Quintela, 2011/06/02
- [Qemu-devel] [PATCH 03/14] flatload: memp was a write-only variable, Juan Quintela, 2011/06/02
- [Qemu-devel] [PATCH 06/14] flatload: end_code was only used in a debug message, Juan Quintela, 2011/06/02
- [Qemu-devel] [PATCH 05/14] linuxload: id_change was a write only variable,
Juan Quintela <=
- [Qemu-devel] [PATCH 07/14] alpha: fn2 was a write only variable, Juan Quintela, 2011/06/02
- [Qemu-devel] [PATCH 08/14] syscall: really return ret code, Juan Quintela, 2011/06/02
- [Qemu-devel] [PATCH 09/14] exec: last_first_tb was only used in !ONLY_USER case, Juan Quintela, 2011/06/02
- [Qemu-devel] [PATCH 11/14] linux-user: fpu_save_addr is not used, Juan Quintela, 2011/06/02