ACPI Thinkpad: We must always call va_end() after va_start() but do not do so in...
authorJesper Juhl <jj@chaosbits.net>
Fri, 24 Dec 2010 18:56:28 +0000 (19:56 +0100)
committerMatthew Garrett <mjg@redhat.com>
Fri, 7 Jan 2011 22:26:25 +0000 (17:26 -0500)
Hi,

In drivers/platform/x86/thinkpad_acpi.c::acpi_evalf() we don't always call
va_end() after va_start(). This patch corrects that.

Signed-off-by: Jesper Juhl <jj@chaosbits.net>
Acked-by: Henrique de Moraes Holschuh <hmh@hmh.eng.br>
Signed-off-by: Matthew Garrett <mjg@redhat.com>
drivers/platform/x86/thinkpad_acpi.c

index f372dc7ae14e478b4e5d258cac7d9842354fe1b8..a974ca383cb90fd1872e36c873006c52be1ab5e0 100644 (file)
@@ -589,6 +589,7 @@ static int acpi_evalf(acpi_handle handle,
                default:
                        printk(TPACPI_ERR "acpi_evalf() called "
                               "with invalid format character '%c'\n", c);
+                       va_end(ap);
                        return 0;
                }
        }