Re: [PATCH] android: binder: Sanity check at binder ioctl

From: kbuild test robot
Date: Mon Jan 18 2016 - 22:39:55 EST


Hi Chen,

[auto build test ERROR on v4.4-rc8]
[also build test ERROR on next-20160118]
[if your patch is applied to the wrong git tree, please drop us a note to help improving the system]

url: https://github.com/0day-ci/linux/commits/Chen-Feng/android-binder-Sanity-check-at-binder-ioctl/20160119-112456
config: x86_64-randconfig-x009-201603 (attached as .config)
reproduce:
# save the attached .config to linux build tree
make ARCH=x86_64

All error/warnings (new ones prefixed by >>):

In file included from include/asm-generic/bug.h:4:0,
from arch/x86/include/asm/bug.h:35,
from include/linux/bug.h:4,
from include/linux/mmdebug.h:4,
from include/linux/mm.h:8,
from include/asm-generic/cacheflush.h:5,
from arch/x86/include/asm/cacheflush.h:5,
from drivers/android/binder.c:20:
drivers/android/binder.c: In function 'binder_ioctl':
>> drivers/android/binder.c:2739:39: error: 'struct task_struct' has no member named 'tsk'
if (unlikely(proc->tsk->mm != current->tsk->mm && proc->tsk->mm))
^
include/linux/compiler.h:147:28: note: in definition of macro '__trace_if'
if (__builtin_constant_p((cond)) ? !!(cond) : \
^
>> drivers/android/binder.c:2739:2: note: in expansion of macro 'if'
if (unlikely(proc->tsk->mm != current->tsk->mm && proc->tsk->mm))
^
>> drivers/android/binder.c:2739:6: note: in expansion of macro 'unlikely'
if (unlikely(proc->tsk->mm != current->tsk->mm && proc->tsk->mm))
^
>> drivers/android/binder.c:2739:39: error: 'struct task_struct' has no member named 'tsk'
if (unlikely(proc->tsk->mm != current->tsk->mm && proc->tsk->mm))
^
include/linux/compiler.h:147:28: note: in definition of macro '__trace_if'
if (__builtin_constant_p((cond)) ? !!(cond) : \
^
>> drivers/android/binder.c:2739:2: note: in expansion of macro 'if'
if (unlikely(proc->tsk->mm != current->tsk->mm && proc->tsk->mm))
^
>> drivers/android/binder.c:2739:6: note: in expansion of macro 'unlikely'
if (unlikely(proc->tsk->mm != current->tsk->mm && proc->tsk->mm))
^
>> drivers/android/binder.c:2739:39: error: 'struct task_struct' has no member named 'tsk'
if (unlikely(proc->tsk->mm != current->tsk->mm && proc->tsk->mm))
^
include/linux/compiler.h:147:28: note: in definition of macro '__trace_if'
if (__builtin_constant_p((cond)) ? !!(cond) : \
^
>> drivers/android/binder.c:2739:2: note: in expansion of macro 'if'
if (unlikely(proc->tsk->mm != current->tsk->mm && proc->tsk->mm))
^
include/linux/compiler.h:123:14: note: in expansion of macro 'likely_notrace'
______r = likely_notrace(x); \
^
include/linux/compiler.h:137:58: note: in expansion of macro '__branch_check__'
# define unlikely(x) (__builtin_constant_p(x) ? !!(x) : __branch_check__(x, 0))
^
>> drivers/android/binder.c:2739:6: note: in expansion of macro 'unlikely'
if (unlikely(proc->tsk->mm != current->tsk->mm && proc->tsk->mm))
^
>> drivers/android/binder.c:2739:39: error: 'struct task_struct' has no member named 'tsk'
if (unlikely(proc->tsk->mm != current->tsk->mm && proc->tsk->mm))
^
include/linux/compiler.h:147:40: note: in definition of macro '__trace_if'
if (__builtin_constant_p((cond)) ? !!(cond) : \
^
>> drivers/android/binder.c:2739:2: note: in expansion of macro 'if'
if (unlikely(proc->tsk->mm != current->tsk->mm && proc->tsk->mm))
^
>> drivers/android/binder.c:2739:6: note: in expansion of macro 'unlikely'
if (unlikely(proc->tsk->mm != current->tsk->mm && proc->tsk->mm))
^
>> drivers/android/binder.c:2739:39: error: 'struct task_struct' has no member named 'tsk'
if (unlikely(proc->tsk->mm != current->tsk->mm && proc->tsk->mm))
^
include/linux/compiler.h:147:40: note: in definition of macro '__trace_if'
if (__builtin_constant_p((cond)) ? !!(cond) : \
^
>> drivers/android/binder.c:2739:2: note: in expansion of macro 'if'
if (unlikely(proc->tsk->mm != current->tsk->mm && proc->tsk->mm))
^
>> drivers/android/binder.c:2739:6: note: in expansion of macro 'unlikely'
if (unlikely(proc->tsk->mm != current->tsk->mm && proc->tsk->mm))
^
>> drivers/android/binder.c:2739:39: error: 'struct task_struct' has no member named 'tsk'
if (unlikely(proc->tsk->mm != current->tsk->mm && proc->tsk->mm))
^
include/linux/compiler.h:147:40: note: in definition of macro '__trace_if'
if (__builtin_constant_p((cond)) ? !!(cond) : \
^
>> drivers/android/binder.c:2739:2: note: in expansion of macro 'if'
if (unlikely(proc->tsk->mm != current->tsk->mm && proc->tsk->mm))
^
include/linux/compiler.h:123:14: note: in expansion of macro 'likely_notrace'
______r = likely_notrace(x); \
^
include/linux/compiler.h:137:58: note: in expansion of macro '__branch_check__'
# define unlikely(x) (__builtin_constant_p(x) ? !!(x) : __branch_check__(x, 0))
^
>> drivers/android/binder.c:2739:6: note: in expansion of macro 'unlikely'
if (unlikely(proc->tsk->mm != current->tsk->mm && proc->tsk->mm))
^
>> drivers/android/binder.c:2739:39: error: 'struct task_struct' has no member named 'tsk'
if (unlikely(proc->tsk->mm != current->tsk->mm && proc->tsk->mm))
^
include/linux/compiler.h:158:16: note: in definition of macro '__trace_if'
______r = !!(cond); \
^
>> drivers/android/binder.c:2739:2: note: in expansion of macro 'if'
if (unlikely(proc->tsk->mm != current->tsk->mm && proc->tsk->mm))
^

vim +2739 drivers/android/binder.c

2733 struct binder_thread *thread;
2734 unsigned int size = _IOC_SIZE(cmd);
2735 void __user *ubuf = (void __user *)arg;
2736
2737 /*pr_info("binder_ioctl: %d:%d %x %lx\n",
2738 proc->pid, current->pid, cmd, arg);*/
> 2739 if (unlikely(proc->tsk->mm != current->tsk->mm && proc->tsk->mm))
2740 return -EINVAL;
2741
2742 trace_binder_ioctl(cmd, arg);

---
0-DAY kernel test infrastructure Open Source Technology Center
https://lists.01.org/pipermail/kbuild-all Intel Corporation

Attachment: .config.gz
Description: Binary data