Hi Eliot,


I really appreciate this patch. It's exactly what I was looking for. I must be 
doing something wrong though. When I apply the patch and try to "make" I get 
this error:



m_options.c: In function âvgPlain_expand_file_nameâ:

m_options.c:244: warning: assignment discards qualifiers from pointer target 
type

m_options.c:252: error: assignment of read-only location â*(format + (long 
unsigned int)(long unsigned int)i)â

m_options.c:253: warning: pointer targets in passing argument 1 of 
âvgPlain_strlenâ differ in signedness

m_options.c:254: warning: pointer targets in passing argument 1 of 
âvgPlain_strcpyâ differ in signedness

m_options.c:254: warning: pointer targets in passing argument 2 of 
âvgPlain_strcpyâ differ in signedness

m_options.c:256: error: assignment of read-only location â*(format + (long 
unsigned int)(long unsigned int)i)â

m_options.c:279: warning: pointer targets in initialization differ in signedness

m_options.c:279: warning: pointer targets in initialization differ in signedness

m_options.c:280: warning: pointer targets in passing argument 1 of 
âvgPlain_execvâ differ in signedness

m_options.c:280: warning: passing argument 2 of âvgPlain_execvâ from 
incompatible pointer type

m_options.c:327: warning: implicit declaration of function âvgPlain_sigemptysetâ

m_options.c:330: warning: implicit declaration of function âvgPlain_sigactionâ

m_options.c:335: warning: implicit declaration of function 
âvgPlain_convert_sigaction_fromK_to_toKâ

m_options.c:346: warning: pointer targets in passing argument 1 of 
âvgPlain_strlenâ differ in signedness

m_options.c:346: warning: pointer targets in passing argument 1 of 
âvgPlain_strlenâ differ in signedness



Line 252 is:

format[i] = 0;

Line 256 is:

      format[i] = '}';





Maybe I didn't apply the patch right...



Kind Regards,

Costas



------------------------------



Date: Tue, 23 Sep 2014 14:11:01 -0400

From: Eliot Moss <m...@cs.umass.edu<mailto:m...@cs.umass.edu>>

Subject: Re: [Valgrind-users] Unique log filename

To: 
"valgrind-users@lists.sourceforge.net<mailto:valgrind-users@lists.sourceforge.net>"

      
<valgrind-users@lists.sourceforge.net<mailto:valgrind-users@lists.sourceforge.net>>

Message-ID: 
<5421b7b5.1030...@cs.umass.edu<mailto:5421b7b5.1030...@cs.umass.edu>>

Content-Type: text/plain; charset="windows-1252"



Ok, here's the patch to add the %s{script} form

to log-file expansion.  The script is run via

/bin/bash -c script, and its output replaces the

while %s form.  The max length my code allows

for the result is 500 characters, which seemed

enough for anything realistic.  Note that the

pid of the invoking valgrind will be the script's

PPID, etc.



If the "powers that be" think this might be

generally useful, then let me know how you would

prefer to receive the patch, and any tidying you

feel would be in order ...



Regards -- Eliot Moss

------------------------------------------------------------------------------
Meet PCI DSS 3.0 Compliance Requirements with EventLog Analyzer
Achieve PCI DSS 3.0 Compliant Status with Out-of-the-box PCI DSS Reports
Are you Audit-Ready for PCI DSS 3.0 Compliance? Download White paper
Comply to PCI DSS 3.0 Requirement 10 and 11.5 with EventLog Analyzer
http://pubads.g.doubleclick.net/gampad/clk?id=154622311&iu=/4140/ostg.clktrk
_______________________________________________
Valgrind-users mailing list
Valgrind-users@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/valgrind-users

Reply via email to