Message ID | 1452502446-11859-1-git-send-email-chengang@emindsoft.com.cn |
---|---|
State | New |
Headers | show |
Le 11/01/2016 09:54, chengang@emindsoft.com.cn a écrit : > From: Chen Gang <chengang@emindsoft.com.cn> > > Implement them according to the other features implementations. > > Signed-off-by: Chen Gang <gang.chen.5i5j@gmail.com> > --- > linux-user/syscall.c | 27 +++++++++++++++++++++++++-- > 1 file changed, 25 insertions(+), 2 deletions(-) > > diff --git a/linux-user/syscall.c b/linux-user/syscall.c > index 44485f2..4c68800 100644 > --- a/linux-user/syscall.c > +++ b/linux-user/syscall.c > @@ -1687,6 +1687,7 @@ static abi_long do_getsockopt(int sockfd, int level, int optname, > abi_long ret; > int len, val; > socklen_t lv; > + struct timeval tv; > > switch(level) { > case TARGET_SOL_SOCKET: > @@ -1694,10 +1695,32 @@ static abi_long do_getsockopt(int sockfd, int level, int optname, > switch (optname) { > /* These don't just return a single integer */ > case TARGET_SO_LINGER: > - case TARGET_SO_RCVTIMEO: > - case TARGET_SO_SNDTIMEO: > case TARGET_SO_PEERNAME: > goto unimplemented; > + case TARGET_SO_RCVTIMEO: > + optname = SO_RCVTIMEO; > + goto time_case; > + case TARGET_SO_SNDTIMEO: > + optname = SO_SNDTIMEO; > + time_case: > + if (get_user_u32(len, optlen)) { > + return -TARGET_EFAULT; > + } > + if (len < sizeof(struct target_timeval)) { > + return -TARGET_EINVAL; > + } Check len >= 0. > + lv = sizeof(tv); > + ret = get_errno(getsockopt(sockfd, level, optname, &tv, &lv)); > + if (ret < 0) { > + return ret; > + } > + if (copy_to_user_timeval(optval_addr, &tv)) { > + return -TARGET_EFAULT; > + } > + if (put_user_u32(sizeof(struct target_timeval), optlen)) { Put in optlen the result of getsockopt(), i.e. "lv", or check lv == sizeof(struct target_timeval). > + return -TARGET_EFAULT; > + } > + break; > case TARGET_SO_PEERCRED: { > struct ucred cr; > socklen_t crlen; >
On 1/11/16 19:04, Peter Maydell wrote: > On 11 January 2016 at 10:31, Chen Gang <chengang@emindsoft.com.cn> wrote: > [various things about a patch] > > Why have we dropped qemu-devel from this email thread? > Oh, sorry, I did not notice about it. Thanks.
diff --git a/linux-user/syscall.c b/linux-user/syscall.c index 44485f2..4c68800 100644 --- a/linux-user/syscall.c +++ b/linux-user/syscall.c @@ -1687,6 +1687,7 @@ static abi_long do_getsockopt(int sockfd, int level, int optname, abi_long ret; int len, val; socklen_t lv; + struct timeval tv; switch(level) { case TARGET_SOL_SOCKET: @@ -1694,10 +1695,32 @@ static abi_long do_getsockopt(int sockfd, int level, int optname, switch (optname) { /* These don't just return a single integer */ case TARGET_SO_LINGER: - case TARGET_SO_RCVTIMEO: - case TARGET_SO_SNDTIMEO: case TARGET_SO_PEERNAME: goto unimplemented; + case TARGET_SO_RCVTIMEO: + optname = SO_RCVTIMEO; + goto time_case; + case TARGET_SO_SNDTIMEO: + optname = SO_SNDTIMEO; + time_case: + if (get_user_u32(len, optlen)) { + return -TARGET_EFAULT; + } + if (len < sizeof(struct target_timeval)) { + return -TARGET_EINVAL; + } + lv = sizeof(tv); + ret = get_errno(getsockopt(sockfd, level, optname, &tv, &lv)); + if (ret < 0) { + return ret; + } + if (copy_to_user_timeval(optval_addr, &tv)) { + return -TARGET_EFAULT; + } + if (put_user_u32(sizeof(struct target_timeval), optlen)) { + return -TARGET_EFAULT; + } + break; case TARGET_SO_PEERCRED: { struct ucred cr; socklen_t crlen;