commit 5d59a6f1ba90f15132d55d9d4d5a632f15a43d84
author Stefan Richter <stefanr@s5r6.in-berlin.de> Tue Jul 17 02:13:48 2007 +0200
committer Stefan Richter <stefanr@s5r6.in-berlin.de> Wed Jul 18 23:53:27 2007 +0200
tree 41482875bd19d9568f2f3af7be8d912f05e5e2fd
parent b980f5a224f3df6c884dbf5ae48797ce352ba139

firewire: remove bogus check in fw_core_handle_request

This check is bogus:
  - Maximum asynchronous payload size for S800...S3200 is 4096.
  - The p->payload_length is totally uninteresting.  Only the
    request->length of the subsequently allocated and initialized
    struct fw_request is of significance.

Signed-off-by: Stefan Richter <stefanr@s5r6.in-berlin.de>
Signed-off-by: Kristian Høgsberg <krh@redhat.com>

drivers/firewire/fw-transaction.c

diff --git a/drivers/firewire/fw-transaction.c b/drivers/firewire/fw-transaction.c
index 80d0121..c4b3417 100644
--- a/drivers/firewire/fw-transaction.c
+++ b/drivers/firewire/fw-transaction.c
@@ -628,11 +628,6 @@
 	unsigned long flags;
 	int tcode, destination, source;
 
-	if (p->payload_length > 2048) {
-		/* FIXME: send error response. */
-		return;
-	}
-
 	if (p->ack != ACK_PENDING && p->ack != ACK_COMPLETE)
 		return;