Project

General

Profile

Actions

Bug #785

closed

fixes for src/sys/bus/firewire/firewire.c

Added by andrew_atrens about 14 years ago. Updated about 14 years ago.

Status:
Closed
Priority:
Normal
Assignee:
-
Category:
-
Target version:
-
Start date:
Due date:
% Done:

0%

Estimated time:

Description

taken from FreeBSD

Index: firewire.c ===================================================================
RCS file: /usr/dfly/repo/src/sys/bus/firewire/firewire.c,v
retrieving revision 1.18
diff u -r1.18 firewire.c
--
firewire.c 22 Dec 2006 23:12:16 0000 1.18
++ firewire.c 1 May 2007 15:09:51 -0000
@ -519,6 +519,7 @
STAILQ_REMOVE_HEAD(&xferq
>q, link);
xferq->queued --;
xfer->resp = EAGAIN;
xfer->state = FWXF_SENTERR;
fw_xfer_done(xfer);
}
}
@ -974,10 +975,10 @
struct fw_xfer *xfer;

xfer = fw_xfer_alloc(type);
- xfer->send.pay_len = send_len;
- xfer->recv.pay_len = recv_len;
if (xfer == NULL)
return(NULL);
+ xfer->send.pay_len = send_len;
+ xfer->recv.pay_len = recv_len;
if (send_len > 0) {
xfer->send.payload = kmalloc(send_len, type, M_INTWAIT | M_ZERO);
if (xfer->send.payload == NULL) {
Actions #1

Updated by dillon about 14 years ago

:taken from FreeBSD
:
:Index: firewire.c
:===================================================================
:RCS file: /usr/dfly/repo/src/sys/bus/firewire/firewire.c,v
:retrieving revision 1.18
:diff u -r1.18 firewire.c
:--
firewire.c 22 Dec 2006 23:12:16 0000 1.18
:+++ firewire.c 1 May 2007 15:09:51 -0000
:@ -519,6 +519,7 @
: STAILQ_REMOVE_HEAD(&xferq
>q, link);
: xferq->queued --;
: xfer->resp = EAGAIN;
:+ xfer->state = FWXF_SENTERR;
: fw_xfer_done(xfer);

Committed.
The xfer allocs can't return NULL so I didn't commit the other bit.
We could just remove all the NULL checks entirely, in fact.
-Matt
Actions #2

Updated by hasso about 14 years ago

It's committed, so closing bug.

Actions

Also available in: Atom PDF