linux-user: untie syscalls from UID16

Quite a number of uid/gid related syscalls are only defined on systems
with USE_UID16 defined. This is apperently based on the idea that these
system calls would never be called on non-UID16 systems. Make these
syscalls available for all architectures that define them.

drop alpha hack to support selected UID16 syscalls. MIPS and PowerPC
were also defined as UID16, to get uid/gid syscalls available, drop
this error as well.

Change QEMU to reflect this.

Cc: Ulrich Hecht <uli@suse.de>
Cc: Richard Henderson <rth@twiddle.net>
Cc: Alexander Graf <agraf@suse.de>
Signed-off-by: Riku Voipio <riku.voipio@iki.fi>
This commit is contained in:
Riku Voipio 2011-04-18 15:23:06 +03:00
parent 42a39fbe0c
commit 0c866a7ed4
3 changed files with 43 additions and 17 deletions

View File

@ -412,10 +412,3 @@
#define TARGET_NR_timerfd 477 #define TARGET_NR_timerfd 477
#define TARGET_NR_eventfd 478 #define TARGET_NR_eventfd 478
/* The following aliases are defined in order to match up with the
standard i386 syscalls implemented in syscalls.c. */
#define TARGET_NR_chown32 TARGET_NR_chown
#define TARGET_NR_setuid32 TARGET_NR_setuid
#define TARGET_NR_setgid32 TARGET_NR_setgid
#define TARGET_NR_setfsuid32 TARGET_NR_setfsuid
#define TARGET_NR_setfsgid32 TARGET_NR_setfsgid

View File

@ -328,7 +328,7 @@ static int sys_fchmodat(int dirfd, const char *pathname, mode_t mode)
return (fchmodat(dirfd, pathname, mode, 0)); return (fchmodat(dirfd, pathname, mode, 0));
} }
#endif #endif
#if defined(TARGET_NR_fchownat) && defined(USE_UID16) #if defined(TARGET_NR_fchownat)
static int sys_fchownat(int dirfd, const char *pathname, uid_t owner, static int sys_fchownat(int dirfd, const char *pathname, uid_t owner,
gid_t group, int flags) gid_t group, int flags)
{ {
@ -437,7 +437,7 @@ _syscall3(int,sys_faccessat,int,dirfd,const char *,pathname,int,mode)
#if defined(TARGET_NR_fchmodat) && defined(__NR_fchmodat) #if defined(TARGET_NR_fchmodat) && defined(__NR_fchmodat)
_syscall3(int,sys_fchmodat,int,dirfd,const char *,pathname, mode_t,mode) _syscall3(int,sys_fchmodat,int,dirfd,const char *,pathname, mode_t,mode)
#endif #endif
#if defined(TARGET_NR_fchownat) && defined(__NR_fchownat) && defined(USE_UID16) #if defined(TARGET_NR_fchownat) && defined(__NR_fchownat)
_syscall5(int,sys_fchownat,int,dirfd,const char *,pathname, _syscall5(int,sys_fchownat,int,dirfd,const char *,pathname,
uid_t,owner,gid_t,group,int,flags) uid_t,owner,gid_t,group,int,flags)
#endif #endif
@ -4164,7 +4164,31 @@ static inline int low2highgid(int gid)
else else
return gid; return gid;
} }
static inline int tswapid(int id)
{
return tswap16(id);
}
#else /* !USE_UID16 */
static inline int high2lowuid(int uid)
{
return uid;
}
static inline int high2lowgid(int gid)
{
return gid;
}
static inline int low2highuid(int uid)
{
return uid;
}
static inline int low2highgid(int gid)
{
return gid;
}
static inline int tswapid(int id)
{
return tswap32(id);
}
#endif /* USE_UID16 */ #endif /* USE_UID16 */
void syscall_init(void) void syscall_init(void)
@ -6765,25 +6789,32 @@ abi_long do_syscall(void *cpu_env, int num, abi_long arg1,
ret = host_to_target_stat64(cpu_env, arg3, &st); ret = host_to_target_stat64(cpu_env, arg3, &st);
break; break;
#endif #endif
#ifdef USE_UID16
case TARGET_NR_lchown: case TARGET_NR_lchown:
if (!(p = lock_user_string(arg1))) if (!(p = lock_user_string(arg1)))
goto efault; goto efault;
ret = get_errno(lchown(p, low2highuid(arg2), low2highgid(arg3))); ret = get_errno(lchown(p, low2highuid(arg2), low2highgid(arg3)));
unlock_user(p, arg1, 0); unlock_user(p, arg1, 0);
break; break;
#ifdef TARGET_NR_getuid
case TARGET_NR_getuid: case TARGET_NR_getuid:
ret = get_errno(high2lowuid(getuid())); ret = get_errno(high2lowuid(getuid()));
break; break;
#endif
#ifdef TARGET_NR_getgid
case TARGET_NR_getgid: case TARGET_NR_getgid:
ret = get_errno(high2lowgid(getgid())); ret = get_errno(high2lowgid(getgid()));
break; break;
#endif
#ifdef TARGET_NR_geteuid
case TARGET_NR_geteuid: case TARGET_NR_geteuid:
ret = get_errno(high2lowuid(geteuid())); ret = get_errno(high2lowuid(geteuid()));
break; break;
#endif
#ifdef TARGET_NR_getegid
case TARGET_NR_getegid: case TARGET_NR_getegid:
ret = get_errno(high2lowgid(getegid())); ret = get_errno(high2lowgid(getegid()));
break; break;
#endif
case TARGET_NR_setreuid: case TARGET_NR_setreuid:
ret = get_errno(setreuid(low2highuid(arg1), low2highuid(arg2))); ret = get_errno(setreuid(low2highuid(arg1), low2highuid(arg2)));
break; break;
@ -6793,7 +6824,7 @@ abi_long do_syscall(void *cpu_env, int num, abi_long arg1,
case TARGET_NR_getgroups: case TARGET_NR_getgroups:
{ {
int gidsetsize = arg1; int gidsetsize = arg1;
uint16_t *target_grouplist; target_id *target_grouplist;
gid_t *grouplist; gid_t *grouplist;
int i; int i;
@ -6806,7 +6837,7 @@ abi_long do_syscall(void *cpu_env, int num, abi_long arg1,
if (!target_grouplist) if (!target_grouplist)
goto efault; goto efault;
for(i = 0;i < ret; i++) for(i = 0;i < ret; i++)
target_grouplist[i] = tswap16(grouplist[i]); target_grouplist[i] = tswapid(high2lowgid(grouplist[i]));
unlock_user(target_grouplist, arg2, gidsetsize * 2); unlock_user(target_grouplist, arg2, gidsetsize * 2);
} }
} }
@ -6814,7 +6845,7 @@ abi_long do_syscall(void *cpu_env, int num, abi_long arg1,
case TARGET_NR_setgroups: case TARGET_NR_setgroups:
{ {
int gidsetsize = arg1; int gidsetsize = arg1;
uint16_t *target_grouplist; target_id *target_grouplist;
gid_t *grouplist; gid_t *grouplist;
int i; int i;
@ -6825,7 +6856,7 @@ abi_long do_syscall(void *cpu_env, int num, abi_long arg1,
goto fail; goto fail;
} }
for(i = 0;i < gidsetsize; i++) for(i = 0;i < gidsetsize; i++)
grouplist[i] = tswap16(target_grouplist[i]); grouplist[i] = low2highgid(tswapid(target_grouplist[i]));
unlock_user(target_grouplist, arg2, 0); unlock_user(target_grouplist, arg2, 0);
ret = get_errno(setgroups(gidsetsize, grouplist)); ret = get_errno(setgroups(gidsetsize, grouplist));
} }
@ -6901,7 +6932,6 @@ abi_long do_syscall(void *cpu_env, int num, abi_long arg1,
case TARGET_NR_setfsgid: case TARGET_NR_setfsgid:
ret = get_errno(setfsgid(arg1)); ret = get_errno(setfsgid(arg1));
break; break;
#endif /* USE_UID16 */
#ifdef TARGET_NR_lchown32 #ifdef TARGET_NR_lchown32
case TARGET_NR_lchown32: case TARGET_NR_lchown32:

View File

@ -49,9 +49,12 @@
#define TARGET_IOC_TYPEBITS 8 #define TARGET_IOC_TYPEBITS 8
#if defined(TARGET_I386) || defined(TARGET_ARM) || defined(TARGET_SPARC) \ #if defined(TARGET_I386) || defined(TARGET_ARM) || defined(TARGET_SPARC) \
|| defined(TARGET_M68K) || defined(TARGET_SH4) || defined(TARGET_CRIS) || defined(TARGET_PPC) || defined(TARGET_MIPS) || defined(TARGET_M68K) || defined(TARGET_SH4) || defined(TARGET_CRIS)
/* 16 bit uid wrappers emulation */ /* 16 bit uid wrappers emulation */
#define USE_UID16 #define USE_UID16
#define target_id uint16_t
#else
#define target_id uint32_t
#endif #endif
#if defined(TARGET_I386) || defined(TARGET_ARM) || defined(TARGET_SH4) \ #if defined(TARGET_I386) || defined(TARGET_ARM) || defined(TARGET_SH4) \