Gitweb:     
http://git.kernel.org/git/?p=linux/kernel/git/torvalds/linux-2.6.git;a=commit;h=5d59a6f1ba90f15132d55d9d4d5a632f15a43d84
Commit:     5d59a6f1ba90f15132d55d9d4d5a632f15a43d84
Parent:     b980f5a224f3df6c884dbf5ae48797ce352ba139
Author:     Stefan Richter <[EMAIL PROTECTED]>
AuthorDate: Tue Jul 17 02:13:48 2007 +0200
Committer:  Stefan Richter <[EMAIL PROTECTED]>
CommitDate: Wed Jul 18 23:53:27 2007 +0200

    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 <[EMAIL PROTECTED]>
    Signed-off-by: Kristian Høgsberg <[EMAIL PROTECTED]>
---
 drivers/firewire/fw-transaction.c |    5 -----
 1 files changed, 0 insertions(+), 5 deletions(-)

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 @@ fw_core_handle_request(struct fw_card *card, struct 
fw_packet *p)
        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;
 
-
To unsubscribe from this list: send the line "unsubscribe git-commits-head" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to