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...@intel.com>
Subject: [PATCH] ima: fix coccinelle warnings
TO: Mimi Zohar <zo...@linux.vnet.ibm.com>
CC: Roberto Sassu <roberto.sa...@polito.it>
CC: linux-kernel@vger.kernel.org 

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.sa...@polito.it>
CC: Mimi Zohar <zo...@linux.vnet.ibm.com>
Signed-off-by: Fengguang Wu <fengguang...@intel.com>
---

 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;

Reply via email to