On Tue, Apr 22, 2014 at 03:08:45PM -0400, pe...@petermalone.org wrote:
Thanks Florian team.
Please review the following diff.
tab vs. space, more in sync with ping6
OK?
diff --git ping.c ping.c
index 6a13a86..29cf84c 100644
--- ping.c
+++ ping.c
@@ -70,6 +70,7 @@
#include ctype.h
#include
Florian Obser flor...@openbsd.org writes:
On Tue, Apr 22, 2014 at 03:08:45PM -0400, pe...@petermalone.org wrote:
Thanks Florian team.
Please review the following diff.
tab vs. space, more in sync with ping6
OK?
ok.
--
jca | PGP : 0x1524E7EE / 5135 92C1 AD36 5293 2BDF DDCC 0DFA 74AE
Sure - I should have spotted that.
Index: ping.c
===
RCS file: /cvs/src/sbin/ping/ping.c,v
retrieving revision 1.100
diff -u -r1.100 ping.c
--- ping.c 24 Mar 2014 11:11:49 - 1.100
+++ ping.c 22 Apr 2014 06:55:03
On Tue, Apr 22, 2014 at 02:57:54AM -0400, pe...@petermalone.org wrote:
Sure - I should have spotted that.
Still not there. Please use the fact that calloc can multiply, you get
an overflow check for free.
-Otto
Index: ping.c
Please switch it to poll(2) like ping6(8) is doing, there by side
stepping the whole issue.
On Tue, Apr 22, 2014 at 09:33:50AM +0200, Otto Moerbeek wrote:
On Tue, Apr 22, 2014 at 02:57:54AM -0400, pe...@petermalone.org wrote:
Sure - I should have spotted that.
Still not there. Please use
On Tue, Apr 22, 2014 at 12:45:25AM -0400, Peter Malone wrote:
Hi,
malloc memset can be replaced with calloc in ping.c. Please see below for
patch details:
Index: ping.c
===
RCS file: /cvs/src/sbin/ping/ping.c,v
retrieving
You are now clearing only the first time.
malloc memset can be replaced with calloc in ping.c. Please see below
for patch details:
Index: ping.c
===
RCS file: /cvs/src/sbin/ping/ping.c,v
retrieving revision 1.100
diff -u
Thanks Florian team.
Please review the following diff.
Index: ping.c
===
RCS file: /cvs/src/sbin/ping/ping.c,v
retrieving revision 1.100
diff -u -p -u -r1.100 ping.c
--- ping.c 24 Mar 2014 11:11:49 - 1.100
+++ ping.c
Hi,
malloc memset can be replaced with calloc in ping.c. Please see below
for patch details:
Index: ping.c
===
RCS file: /cvs/src/sbin/ping/ping.c,v
retrieving revision 1.100
diff -u -p -u -r1.100 ping.c
--- ping.c24 Mar
On Tue, Apr 22, 2014 at 12:45:25AM -0400, Peter Malone wrote:
Hi,
malloc memset can be replaced with calloc in ping.c. Please see below for
patch details:
Better rework this to get rid of fdmasks.
-Otto
Index: ping.c
10 matches
Mail list logo