Re: [PATCH v2 1/2] xattr: use printstr_ex instead of print_quoted_string

2017-03-13 Thread Mike Frysinger
On 23 Dec 2016 08:28, JingPiao Chen wrote: > Could I ask a question? All the string even in struct should comply with -s > option? > All the string should NUL terminate? Thanks. it probably depends on the syscall ... not all buffers are NUL terminated. it would help if you quoted a specific syscal

Re: A: sched_xetattr.test fails consistently on aarch64

2017-03-13 Thread Mike Frysinger
On 14 Mar 2017 00:57, Mike Frysinger wrote: > so there's a disagreement in the overall system somewhere. > i'd assume either the kernel's implementation of access_ok, > or gcc's handling of the inline assembly. > > once i have access to a local system and reduce a bit more, > i'll file some upstre

Re: A: sched_xetattr.test fails consistently on aarch64

2017-03-13 Thread Mike Frysinger
On 14 Mar 2017 00:13, Dmitry V. Levin wrote: > sched_xetattr.test introduced by commit v4.16-10-gf31755f > (tests: check decoding of sched_[gs]etattr corner cases) > consistently fails on aarch64. > > At the same time the test passes on all other architectures > where strace is being tested regula

Re: GSOC: Introduction

2017-03-13 Thread Abhishek Tiwari
On Tue, Mar 14, 2017 at 2:00 AM, Dmitry V. Levin wrote: > On Tue, Mar 14, 2017 at 12:40:46AM +0530, Abhishek Tiwari wrote: >> > Per-file summary is not in GNU change log format, please refer to >> > https://www.gnu.org/prep/standards/html_node/Style-of-Change-Logs.html . >> > Specifically, it lack

Re: Implemented -e trace=%clock option

2017-03-13 Thread Dmitry V. Levin
On Mon, Mar 13, 2017 at 11:20:30PM +0530, Rishi Bhatt wrote: > Hey does $STRACE in init.sh is doing this:strace -e trace=%clock > ./clock_nanosleep > /dev/null > > So my problem here is the output in .tmp file is correct but the .exp has > the print statements from ./clock_nanosleep so for matchi

Re: A: sched_xetattr.test fails consistently on aarch64

2017-03-13 Thread Dmitry V. Levin
On Mon, Mar 13, 2017 at 08:04:02PM -0400, Mike Frysinger wrote: > On 14 Mar 2017 00:13, Dmitry V. Levin wrote: > > However, I'm not an aarch64 expert myself, nor do I have the appropriate > > aarch64 hardware to investigate, so this issue is not going to be solved > > without your help. > > fwiw,

Re: Gsoc Introduction.

2017-03-13 Thread Dmitry V. Levin
Hi, On Tue, Mar 14, 2017 at 08:35:45AM +0700, Николай Марчук wrote: > Hello, > my name is Nikolay Marchuk and I am a first-year student in NSU, Russia. I > want to implement advanced filtering syntax for strace. I am familiar with > c and c++ and have an experience of making parsers. Thanks for s

Gsoc Introduction.

2017-03-13 Thread Николай Марчук
Hello, my name is Nikolay Marchuk and I am a first-year student in NSU, Russia. I want to implement advanced filtering syntax for strace. I am familiar with c and c++ and have an experience of making parsers. I wanted to add support of new ioctl commands from linux 4.9 - 4.11 as my introduction mic

Re: A: sched_xetattr.test fails consistently on aarch64

2017-03-13 Thread Mike Frysinger
On 14 Mar 2017 00:13, Dmitry V. Levin wrote: > However, I'm not an aarch64 expert myself, nor do I have the appropriate > aarch64 hardware to investigate, so this issue is not going to be solved > without your help. fwiw, i should have a Gentoo system available for you to remote access in the next

A: sched_xetattr.test fails consistently on aarch64

2017-03-13 Thread Dmitry V. Levin
Hi, sched_xetattr.test introduced by commit v4.16-10-gf31755f (tests: check decoding of sched_[gs]etattr corner cases) consistently fails on aarch64. At the same time the test passes on all other architectures where strace is being tested regularly, both 64-bit (alpha, ia64, mips64, ppc64, s390x,

Re: GSOC: Introduction

2017-03-13 Thread Dmitry V. Levin
On Tue, Mar 14, 2017 at 12:40:46AM +0530, Abhishek Tiwari wrote: > > Per-file summary is not in GNU change log format, please refer to > > https://www.gnu.org/prep/standards/html_node/Style-of-Change-Logs.html . > > Specifically, it lacks asterisks at the beginning of each new file > > description

Re: Implemented -e trace=%clock option

2017-03-13 Thread Rishi Bhatt
Hey does $STRACE in init.sh is doing this:strace -e trace=%clock ./clock_nanosleep > /dev/null So my problem here is the output in .tmp file is correct but the .exp has the print statements from ./clock_nanosleep so for matching .exp and .tmp files i have to filter setitimer string from .exp file

Re: GSoC: Introduction

2017-03-13 Thread Dmitry V. Levin
Hi, On Sun, Mar 12, 2017 at 06:55:44AM +0100, Israel vianney wrote: > Hi, > > I am Vianney Israel a second year undergraduate computer science student in > ISICOM University from Cameroon. I am new to open-source and i wish to > contribute to Strace as for Gsoc 2017. > I am familiar with C, git a

Re: [PATCH v1] unwind.c: fix a possible buffer overflow

2017-03-13 Thread Dmitry V. Levin
On Sat, Mar 11, 2017 at 02:27:33PM +0300, Victor Krapivensky wrote: > Linux does not prevent a user from creating a lot of nested directories > with length of the absolute path of the deepest one exceeding PATH_MAX, > then chdir'ing into it, creating a file there and mmap'ing it. Since the > length

Re: [PATCH v1] util.c: remove a wrong comment

2017-03-13 Thread Dmitry V. Levin
On Fri, Mar 10, 2017 at 08:48:35PM +0300, Victor Krapivensky wrote: > 4924dbd6d750665cf383b20ab4fd67e48219ab91 modified the return type, but > not the comment. > --- > util.c | 1 - > 1 file changed, 1 deletion(-) > > diff --git a/util.c b/util.c > index a38f821..f05d4ef 100644 > --- a/util.c > +