diff for duplicates of <5034CD80.70501@kmckk.co.jp> diff --git a/a/1.txt b/N1/1.txt index 3dcaa1e..ed478d4 100644 Binary files a/a/1.txt and b/N1/1.txt differ diff --git a/a/content_digest b/N1/content_digest index 579ad70..3458ad3 100644 --- a/a/content_digest +++ b/N1/content_digest @@ -14,7 +14,7 @@ "ref\0Pine.LNX.4.64.1208220840400.26767@axis700.grange\0" "From\0Tetsuyuki Kobayashi <koba@kmckk.co.jp>\0" "Subject\0Re: [PATCH] mmc: sh-mmcif: avoid Oops on spurious interrupts\0" - "Date\0Wed, 22 Aug 2012 21:16:00 +0900\0" + "Date\0Wed, 22 Aug 2012 12:16:00 +0000\0" "To\0Guennadi Liakhovetski <g.liakhovetski@gmx.de>\0" "Cc\0yusuke.goda.sx@renesas.com" Kuninori Morimoto <kuninori.morimoto.gx@renesas.com> @@ -57,7 +57,7 @@ ">>\n" ">> Unable to handle kernel NULL pointer dereference at virtual address 00000008\n" ">> pgd = c0004000\n" - ">> [00000008] *pgd=00000000\n" + ">> [00000008] *pgd\0000000\n" ">> Internal error: Oops: 17 [#1] PREEMPT SMP ARM\n" ">> Modules linked in:\n" ">> CPU: 1 Not tainted (3.6.0-rc2+ #103)\n" @@ -81,8 +81,8 @@ ">> + struct mmc_data *data;\n" ">> +\n" ">> + /* quick fix by koba */\n" - ">> + if (mrq == NULL) {\n" - ">> + printk(\"sh_mmcif_irqt: mrq == NULL: host->wait_for=%d\\n\", host->wait_for);\n" + ">> + if (mrq = NULL) {\n" + ">> + printk(\"sh_mmcif_irqt: mrq = NULL: host->wait_for=%d\\n\", host->wait_for);\n" ">> + } else {\n" ">> + data = mrq->data;\n" ">> + }\n" @@ -92,8 +92,8 @@ ">>\n" ">> With this patch, there is no null pointer accesses and got this log.\n" ">>\n" - ">> sh_mmcif_irqt: mrq == NULL: host->wait_for=0\n" - ">> sh_mmcif_irqt: mrq == NULL: host->wait_for=0\n" + ">> sh_mmcif_irqt: mrq = NULL: host->wait_for=0\n" + ">> sh_mmcif_irqt: mrq = NULL: host->wait_for=0\n" ">> ...\n" ">>\n" ">> host->wait_for is 0. it is MMCIF_WAIT_FOR_REQUEST.\n" @@ -102,7 +102,7 @@ ">> host->wait_for = MMCIF_WAIT_FOR_REQUEST;\n" ">> host->mrq = NULL;\n" ">>\n" - ">> So, at the top of sh_mmcif_irqt, if host->wait_for == MMCIF_WAIT_FOR_REQUEST,\n" + ">> So, at the top of sh_mmcif_irqt, if host->wait_for = MMCIF_WAIT_FOR_REQUEST,\n" ">> host->mrq = NULL.\n" ">> It is too earlier to access mrq->data before checking host->mrq. it may\n" ">> cause null pointer access.\n" @@ -150,8 +150,7 @@ "> \tif (host->wait_for != MMCIF_WAIT_FOR_STOP) {\n" "> +\t\tstruct mmc_data *data = mrq->data;\n" "> \t\tif (!mrq->cmd->error && data && !data->error)\n" - "> \t\t\tdata->bytes_xfered =\n" - "> \t\t\t\tdata->blocks * data->blksz;\n" + "> \t\t\tdata->bytes_xfered > \t\t\t\tdata->blocks * data->blksz;\n" > -2abb6d8685279a7837c3f0bf42e830d2d0ea063115e76f7a6466199a476b7f6b +e1132b9c105f2ac44f8d55d8cf759225d38007c146c76a560fa71fb9117e9ec2
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.