koct9i / ioping

simple disk I/0 latency measuring tool
GNU General Public License v3.0
947 stars 76 forks source link

[ PATCH ] memleak() + incorrect strlens #2

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
1. memleak 
2. incorrect strlen 

Original issue reported on code.google.com by dixlor on 6 Jun 2011 at 10:39

Attachments:

GoogleCodeExporter commented 9 years ago
Yet another patch 

http://www.opennet.ru/openforum/vsluhforumID3/77683.html#27

Original comment by dixlor on 6 Jun 2011 at 11:49

GoogleCodeExporter commented 9 years ago
1. not a leak, buffer will freed automatically at task exit
2. it correct, size("%s%s", a, b) == strlen(a) + strlen(b) + 1

Original comment by koct9i on 7 Jun 2011 at 7:29

GoogleCodeExporter commented 9 years ago
all this mess required just to make valgrind completely happy?

Original comment by koct9i on 7 Jun 2011 at 7:34

GoogleCodeExporter commented 9 years ago
Yeap  :)

Original comment by dixlor on 7 Jun 2011 at 11:45

GoogleCodeExporter commented 9 years ago

Original comment by koct9i on 9 Jun 2011 at 2:56

GoogleCodeExporter commented 9 years ago

Original comment by koct9i on 28 Feb 2013 at 7:53