svn commit: r232723 - head/sys/netinet
Michael Tuexen
tuexen at FreeBSD.org
Fri Mar 9 15:45:02 UTC 2012
On Mar 9, 2012, at 3:08 PM, Bjoern A. Zeeb wrote:
>
> On 9. Mar 2012, at 13:12 , Michael Tuexen wrote:
>
>> Author: tuexen
>> Date: Fri Mar 9 13:12:33 2012
>> New Revision: 232723
>> URL: http://svn.freebsd.org/changeset/base/232723
>>
>> Log:
>> Fix a bug reported by Peter Holm which results in a crash:
>> Verify in sctp_peeloff() that the socket is a one-to-many
>> style SCTP socket.
>
> /scratch/tmp/bz/head.svn/sys/netinet/sctp_peeloff.c: In function 'sctp_can_peel_off':
> /scratch/tmp/bz/head.svn/sys/netinet/sctp_peeloff.c:59: warning: 'inp' is used uninitialized in this function
Fixed in http://svn.freebsd.org/changeset/base/232726
Thanks for the report.
Best regards
Michael
>
>
>>
>> MFC after: 3 days.
>>
>> Modified:
>> head/sys/netinet/sctp_peeloff.c
>>
>> Modified: head/sys/netinet/sctp_peeloff.c
>> ==============================================================================
>> --- head/sys/netinet/sctp_peeloff.c Fri Mar 9 13:06:24 2012 (r232722)
>> +++ head/sys/netinet/sctp_peeloff.c Fri Mar 9 13:12:33 2012 (r232723)
>> @@ -55,6 +55,15 @@ sctp_can_peel_off(struct socket *head, s
>> struct sctp_tcb *stcb;
>> uint32_t state;
>>
>> + if (head == NULL) {
>> + SCTP_LTRACE_ERR_RET(inp, NULL, NULL, SCTP_FROM_SCTP_PEELOFF, EBADF);
>> + return (EBADF);
>> + }
>> + if ((head->so_proto->pr_protocol != IPPROTO_SCTP) ||
>> + (head->so_type != SOCK_SEQPACKET)) {
>> + SCTP_LTRACE_ERR_RET(inp, NULL, NULL, SCTP_FROM_SCTP_PEELOFF, EOPNOTSUPP);
>> + return (EOPNOTSUPP);
>> + }
>> inp = (struct sctp_inpcb *)head->so_pcb;
>> if (inp == NULL) {
>> SCTP_LTRACE_ERR_RET(inp, NULL, NULL, SCTP_FROM_SCTP_PEELOFF, EFAULT);
>
> --
> Bjoern A. Zeeb You have to have visions!
> It does not matter how good you are. It matters what good you do!
>
>
More information about the svn-src-all
mailing list