aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorStefan Richter <stefanr@s5r6.in-berlin.de>2007-07-16 20:13:48 -0400
committerStefan Richter <stefanr@s5r6.in-berlin.de>2007-07-18 17:53:27 -0400
commit5d59a6f1ba90f15132d55d9d4d5a632f15a43d84 (patch)
tree41482875bd19d9568f2f3af7be8d912f05e5e2fd
parentb980f5a224f3df6c884dbf5ae48797ce352ba139 (diff)
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>
-rw-r--r--drivers/firewire/fw-transaction.c5
1 files changed, 0 insertions, 5 deletions
diff --git a/drivers/firewire/fw-transaction.c b/drivers/firewire/fw-transaction.c
index 80d0121463d0..c4b3417a5dac 100644
--- a/drivers/firewire/fw-transaction.c
+++ b/drivers/firewire/fw-transaction.c
@@ -628,11 +628,6 @@ fw_core_handle_request(struct fw_card *card, struct fw_packet *p)
628 unsigned long flags; 628 unsigned long flags;
629 int tcode, destination, source; 629 int tcode, destination, source;
630 630
631 if (p->payload_length > 2048) {
632 /* FIXME: send error response. */
633 return;
634 }
635
636 if (p->ack != ACK_PENDING && p->ack != ACK_COMPLETE) 631 if (p->ack != ACK_PENDING && p->ack != ACK_COMPLETE)
637 return; 632 return;
638 633