src/sys/netgraph/ng_socket.c, branch releng/7.3

MFC rev. 185183

2009-01-09T20:41:58Z

Remove unused variable. Found with: Coverity Prevent(tm) CID: 3682

MFC:

2008-03-16T22:06:07Z

Improve apply callback error reporting: Before this patch callback returned result of the last finished call chain. Now it returns last nonzero result from all call chain results in this request. As soon as this improvement gives reliable error reporting, it is now possible to remove dirty workaround in ng_socket, made to return ENOBUFS error statuses of request-response operations. That workaround was responsible for returning ENOBUFS errors to completely unrelated requests working at the same time on socket.

MFC ng_base.c rev 1.136-1.137, netgraph.h rev 1.69, ng_socket.c rev 1.83

2007-10-28T17:52:08Z

Implement new apply callback mechanism to handle item forwarding. When item forwarded, refence counter is incremented, when item processed, counter decremented. When counter reaches zero, apply handler is getting called. It allows to report connect() call status only when it is really completed. Reviewed by: julian Approved by: re (kensmith)

Reduce network stack oddness: implement .pru_sockaddr and .pru_peeraddr

2007-05-11T10:20:51Z

protocol entry points using functions named proto_getsockaddr and proto_getpeeraddr rather than proto_setsockaddr and proto_setpeeraddr. While it's true that sockaddrs are allocated and set, the net effect is to retrieve (get) the socket address or peer address from a socket, not set it, so align names to that intent.

Sweep kernel replacing suser(9) calls with priv(9) calls, assigning

2006-11-06T13:42:10Z

specific privilege names to a broad range of privileges. These may require some future tweaking. Sponsored by: nCircle Network Security, Inc. Obtained from: TrustedBSD Project Discussed on: arch@ Reviewed (at least in part) by: mlaier, jmg, pjd, bde, ceri, Alex Lyashkov , Skip Ford , Antoine Brodin

Check pointer before dereferencing.

2006-10-18T07:47:07Z

Reported by: Coverity CID: 1556

Some clenaup of ngs_rcvmsg():

2006-10-17T16:52:09Z

- Inline ship_msg() into ngs_rcvmsg(). - Plug memory leak in case if no control socket present. - Remove malloc() and allocate the sockaddr on stack. - style(9).

Some cleanup and small changes:

2006-10-17T12:21:48Z

- Use malloc() and free() instead of MALLOC() and FREE() macros. - Do not check malloc results if M_WAITOK was used. - Remove linked list of all netgraph sockets. It isn't needed. - Use ng_findhook() instead of searching the list ourselves. - Use NG_WAITOK in syscalls. - Remove unneeded includes. - style(9)

Change semantics of socket close and detach. Add a new protocol switch

2006-07-21T17:11:15Z

function, pru_close, to notify protocols that the file descriptor or other consumer of a socket is closing the socket. pru_abort is now a notification of close also, and no longer detaches. pru_detach is no longer used to notify of close, and will be called during socket tear-down by sofree() when all references to a socket evaporate after an earlier call to abort or close the socket. This means detach is now an unconditional teardown of a socket, whereas previously sockets could persist after detach of the protocol retained a reference. This faciliates sharing mutexes between layers of the network stack as the mutex is required during the checking and removal of references at the head of sofree(). With this change, pru_detach can now assume that the mutex will no longer be required by the socket layer after completion, whereas before this was not necessarily true. Reviewed by: gnn

Use kern_kldload() and kern_kldunload() to load and unload modules when

2006-06-13T21:36:23Z

we intend for the user to be able to unload them later via kldunload(2) instead of calling linker_load_module() and then directly adjusting the ref count on the linker file structure. This makes the resulting consumer code simpler and cleaner and better hides the linker internals making it possible to sanely lock the linker.