UbiquitousOS
/
OpenCloudOS-Kernel
mirror of https://gitee.com/OpenCloudOS/OpenCloudOS-Kernel.git
11
0
Fork 0
Code Issues Projects Releases Wiki Activity

firewire: fix memory leak of fw_request instances 

Found and debugged by Jay Fenlason <fenlason@redhat.com>.
The bug was especially noticeable with direct I/O over fw-sbp2.

Signed-off-by: Stefan Richter <stefanr@s5r6.in-berlin.de>
Signed-off-by: Kristian Høgsberg <krh@redhat.com>
This commit is contained in:
Stefan Richter 2007-07-17 02:15:36 +02:00
parent 5d59a6f1ba
commit 9c9bdf4d50
2 changed files with 7 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
drivers/firewire/fw-transaction.c
View File

@ -605,8 +605,10 @@ fw_send_response(struct fw_card *card, struct fw_request *request, int rcode)
* check is sufficient to ensure we don't send response to * check is sufficient to ensure we don't send response to
* broadcast packets or posted writes. * broadcast packets or posted writes.
*/ */
if (request->ack != ACK_PENDING) if (request->ack != ACK_PENDING) {
kfree(request);
return; return;
}
if (rcode == RCODE_COMPLETE) if (rcode == RCODE_COMPLETE)
fw_fill_response(&request->response, request->request_header, fw_fill_response(&request->response, request->request_header,

4
drivers/firewire/fw-transaction.h
View File

@ -123,6 +123,10 @@ typedef void (*fw_transaction_callback_t)(struct fw_card *card, int rcode,
size_t length, size_t length,
void *callback_data); void *callback_data);
/*
* Important note: The callback must guarantee that either fw_send_response()
* or kfree() is called on the @request.
*/
typedef void (*fw_address_callback_t)(struct fw_card *card, typedef void (*fw_address_callback_t)(struct fw_card *card,
struct fw_request *request, struct fw_request *request,
int tcode, int destination, int source, int tcode, int destination, int source,