diff for duplicates of <505932DD.8010703@kmckk.co.jp> diff --git a/a/1.txt b/N1/1.txt index d574b54..9e7f86e 100644 Binary files a/a/1.txt and b/N1/1.txt differ diff --git a/a/content_digest b/N1/content_digest index 3f8c1a3..08d853c 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, 19 Sep 2012 11:50:05 +0900\0" + "Date\0Wed, 19 Sep 2012 02:50:05 +0000\0" "To\0Guennadi Liakhovetski <g.liakhovetski@gmx.de>\0" "Cc\0yusuke.goda.sx@renesas.com" Kuninori Morimoto <kuninori.morimoto.gx@renesas.com> @@ -55,7 +55,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" @@ -79,8 +79,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" @@ -90,8 +90,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" @@ -100,7 +100,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" @@ -148,8 +148,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" > -7fc315ce49aef9a214b34a7a234e36db30fee52d6723300e14046177f07add4e +2db5f8da4626300de74864ec34c2979620aac879b560af11f00d9f0639dd7c79
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.