Re: [BUG] 3ce1217d6cd5 ima patch causes s390 to crash on boot

[Roberto Sassu]

On 11/22/2013 12:48 PM, Heiko Carstens wrote:
> Hi Roberto,
>
> your patch 3ce1217d6cd5 "ima: define template fields library and new helpers"
> causes s390 to crash on boot:

Hi Heiko

thanks for the information. I think this issue is related to the error
detected by the kbuild test robot. Please, try to apply the attached
patch to see if it solves the problem.

Thanks

Roberto


>   IMA: No TPM chip found, activating TPM-bypass!
>   Unable to handle kernel pointer dereference at virtual kernel address 0000000000842000
>   Oops: 0004 [#1] SMP
>   Modules linked in:
>   CPU: 3 PID: 1 Comm: swapper/0 Not tainted 3.12.0-rc2-00098-g3ce1217d6cd5 #17
>   task: 000000003ffa0000 ti: 000000003ff84000 task.ti: 000000003ff84000
>   Krnl PSW : 0704e00180000000 000000000044bf88 (strsep+0x7c/0xa0)
>              R:0 T:1 IO:1 EX:1 Key:0 M:1 W:0 P:0 AS:3 CC:2 PM:0 EA:3
>   Krnl GPRS: 000000000000007c 000000000000007c 000000003ff87d90 0000000000821fd8
>              0000000000000000 000000000000007c 0000000000aa37e0 0000000000aa9008
>              0000000000000051 0000000000a114d8 0000000100000002 0000000000842bde
>              0000000000842bdf 00000000006f97f0 000000000040062c 000000003ff87cf0
>   Krnl Code: 000000000044bf7c: a7f4000a           brc     15,44bf90
>              000000000044bf80: b90200cc           ltgr    %r12,%r12
>             #000000000044bf84: a7840006           brc     8,44bf90
>             >000000000044bf88: 9200c000           mvi     0(%r12),0
>              000000000044bf8c: 41c0c001           la      %r12,1(%r12)
>              000000000044bf90: e3c020000024       stg     %r12,0(%r2)
>              000000000044bf96: b904002b           lgr     %r2,%r11
>              000000000044bf9a: ebbcf0700004       lmg     %r11,%r12,112(%r15)
>   Call Trace:
>   ([<00000000004005fe>] ima_init_template+0xa2/0x1bc)
>    [<0000000000a7c896>] ima_init+0x7a/0xa8
>    [<0000000000a7c938>] init_ima+0x24/0x40
>    [<00000000001000e8>] do_one_initcall+0x68/0x128
>    [<0000000000a4eb56>] kernel_init_freeable+0x20a/0x2b4
>    [<00000000006a1ff4>] kernel_init+0x30/0x178
>    [<00000000006b69fe>] kernel_thread_starter+0x6/0xc
>    [<00000000006b69f8>] kernel_thread_starter+0x0/0xc
>   Last Breaking-Event-Address:
>    [<000000000044bf42>] strsep+0x36/0xa0

From: Fengguang Wu <fengguang.wu@xxxxxxxxx>
Subject: [PATCH] ima: fix coccinelle warnings
TO: Mimi Zohar <zohar@xxxxxxxxxxxxxxxxxx>
CC: Roberto Sassu <roberto.sassu@xxxxxxxxx>
CC: linux-kernel@xxxxxxxxxxxxxxx 

security/integrity/ima/ima_template.c:62:41-47: ERROR: application of sizeof to pointer

 sizeof when applied to a pointer typed expression gives the size of
 the pointer

Generated by: coccinelle/misc/noderef.cocci

CC: Roberto Sassu <roberto.sassu@xxxxxxxxx>
CC: Mimi Zohar <zohar@xxxxxxxxxxxxxxxxxx>
Signed-off-by: Fengguang Wu <fengguang.wu@xxxxxxxxx>
---

 cocci-output-13142-271b5e-ima_template.c |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

--- a/security/integrity/ima/ima_template.c
+++ b/security/integrity/ima/ima_template.c
@@ -59,7 +59,7 @@ static int template_desc_init_fields(cha
 	if (template_num_fields > IMA_TEMPLATE_NUM_FIELDS_MAX)
 		return -EINVAL;
 
-	*fields = kzalloc(template_num_fields * sizeof(*fields), GFP_KERNEL);
+	*fields = kzalloc(template_num_fields * sizeof(**fields), GFP_KERNEL);
 	if (*fields == NULL) {
 		result = -ENOMEM;
 		goto out;