Commit:     8a3a0ee736b9a0150c9d894f2c6cc836a62125a1
Parent:     a76c0b976310bbb1b6eaecaaae465af194134477
Author:     Kenji Kaneshige <[EMAIL PROTECTED]>
AuthorDate: Mon Mar 26 09:38:42 2007 +0900
Committer:  Tony Luck <[EMAIL PROTECTED]>
CommitDate: Thu Mar 29 15:02:58 2007 -0700

    [IA64] Fix possible invalid memory access in ia64_setup_msi_irq()
    The following 'if' statement in ia64_setup_msi_irq() always fails even
    if create_irq() returns <0 value, because variable 'irq' is defined as
    unsigned int. It would cause invalid memory access.
            irq = create_irq();
            if (irq < 0)
                    return irq;
    Signed-off-by: Kenji Kaneshige <[EMAIL PROTECTED]>
    Signed-off-by: Tony Luck <[EMAIL PROTECTED]>
 arch/ia64/kernel/msi_ia64.c |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/arch/ia64/kernel/msi_ia64.c b/arch/ia64/kernel/msi_ia64.c
index e722090..ebbeadf 100644
--- a/arch/ia64/kernel/msi_ia64.c
+++ b/arch/ia64/kernel/msi_ia64.c
@@ -68,7 +68,7 @@ int ia64_setup_msi_irq(struct pci_dev *pdev, struct msi_desc 
        struct msi_msg  msg;
        unsigned long   dest_phys_id;
-       unsigned int    irq, vector;
+       int     irq, vector;
        irq = create_irq();
        if (irq < 0)
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

Reply via email to