Opened 13 years ago

Closed 13 years ago

Last modified 13 years ago

#689 closed bug (fixed)

[ping] 2 small fixes

Reported by: olivier.coursiere@… Owned by: korli
Priority: normal Milestone: R1
Component: - General Version:
Keywords: Cc: diver
Blocked By: Blocking:
Has a Patch: no Platform: All

Description

  • errx's error message should be ended with a new line
    • the type of fromlen in main function should match the type of the last

argument in recvfrom. So, fromlen should be "int" instead of "size_t".

Attachments (1)

ping.diff (410 bytes) - added by olivier.coursiere@… 13 years ago.
patch to fix those problems

Download all attachments as: .zip

Change History (9)

Changed 13 years ago by olivier.coursiere@…

Attachment: ping.diff added

patch to fix those problems

comment:1 Changed 13 years ago by diver

Cc: diver added

comment:2 Changed 13 years ago by korli

Cc: axeld@… added

comment:3 Changed 13 years ago by korli

Thanks for the patch. On the second problem, I think it would be better anyway to use socklen_t in sys/socket.h instead of int. Axel, could changing to socklen_t make Haiku less BeOS compatible, etc.. ?

comment:4 Changed 13 years ago by axeld

It would only affect source compatibility, and I wouldn't have a problem with that (but I don't expect any problems with this either) :-) IOW I'd welcome socklen_t.

comment:5 Changed 13 years ago by korli

I'm wondering : we could only change it for our libnetwork.so and our headers, not the compat net libs and legacy network headers ?

comment:6 Changed 13 years ago by korli

Status: newclosed

comment:7 Changed 13 years ago by korli

Resolution: fixed

comment:8 Changed 13 years ago by korli

Committed the relevant part of the patch, and changed size parameter type of sys/socket.h functions to socklen_t for our own headers and libnetwork.so in revision 17911.

Note: See TracTickets for help on using tickets.