On Tue, Jul 01, 2014 at 03:13:21PM +0530, zubin.mit...@gmail.com wrote:
> * defs.h (show_fd_path): Change type to unsigned int.
> * strace.c (show_fd_path): Update usage to count y flag.
> (usage): Update to add -yy option.
> * util.c (print_tracee_cwd): New function.
> (printpathn): Update to use print_tracee_cwd and print
>  absolute path.
> * strace.1: Add description of -yy option.

I had an opportunity to think about this feature a bit longer,
and now I see two issues I missed before.

The first issue is the choice of option name.  While -y option enables
*additional* decoding of descriptors, and -yy was planned for extended
decoding of descriptors like socket decoding, this feature *changes*
decoding of all paths.  I'd rather not mix these things.

The second issue is *at syscalls.  This implementation assumes that all
non-absolute paths are relative to cwd, but that is not necessarily the
case for *at syscalls.


-- 
ldv

Attachment: pgpvXROgwIfKB.pgp
Description: PGP signature

------------------------------------------------------------------------------
_______________________________________________
Strace-devel mailing list
Strace-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/strace-devel

Reply via email to