drivers/tty/vt/vt_ioctl.c:1004:21: sparse: sparse: incorrect type in argument 1 (different address spaces)

From: kernel test robot
Date: Tue Sep 15 2020 - 01:55:30 EST

Next message:
Previous message: Messages sorted by:
tree: .config.gz< Description: application/gzip Randy Dunlap: "Re: xconfig is broken again on Qt5" href="08422.html"> Yong Wu: "Re: [PATCH v2 01/23] dt-bindings: iommu: mediatek: Convert IOMMU to DT schema" href="date.html#08423">[ date ] [ thread ] [ subject ] [ author ] org/pub/scm/linux/kernel/git/torvalds/linux.git">https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master
sparse: use static inline for __chk_{user,io}_ptr()
as .config)
thubusercontent.com/intel/lkp-tests/master/sbin/make.cross">https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross

9f2e76959
build tree
make.cross C=1 CF='-fdiagnostic-prefix -D__CHECK_ENDIAN__' ARCH=sh
tag as appropriate

sparse: incorrect type in initializer (different address spaces) @@ expected unsigned short const *__gu_addr @@ got unsigned short [noderef] __user * @@
expected unsigned short const *__gu_addr
got unsigned short [noderef] __user *
sparse: sparse: incorrect type in argument 1 (different address spaces) @@ expected void const volatile [noderef] __user *ptr @@ got unsigned short const *__gu_addr @@
expected void const volatile [noderef] __user *ptr
got unsigned short const *__gu_addr
sparse: incorrect type in initializer (different address spaces) @@ expected unsigned short const *__gu_addr @@ got unsigned short [noderef] __user * @@
expected unsigned short const *__gu_addr
got unsigned short [noderef] __user *
sparse: incorrect type in argument 1 (different address spaces) @@ expected void const volatile [noderef] __user *ptr @@ got unsigned short const *__gu_addr @@
expected void const volatile [noderef] __user *ptr
got unsigned short const *__gu_addr
incorrect type in initializer (different address spaces) @@ expected char const *__gu_addr @@ got char [noderef] __user * @@
expected char const *__gu_addr
got char [noderef] __user *
sparse: incorrect type in argument 1 (different address spaces) @@ expected void const volatile [noderef] __user *ptr @@ got char const *__gu_addr @@
expected void const volatile [noderef] __user *ptr
got char const *__gu_addr
symbol 'console_blank_hook' was not declared. Should it be static?
symbol 'console_driver' was not declared. Should it be static?
incorrect type in initializer (different address spaces) @@ expected char const *__gu_addr @@ got char [noderef] __user *p @@
expected char const *__gu_addr
got char [noderef] __user *p
incorrect type in argument 1 (different address spaces) @@ expected void const volatile [noderef] __user *ptr @@ got char const *__gu_addr @@
expected void const volatile [noderef] __user *ptr
got char const *__gu_addr
incorrect type in initializer (different address spaces) @@ expected char const *__gu_addr @@ got char [noderef] __user * @@
expected char const *__gu_addr
got char [noderef] __user *
incorrect type in argument 1 (different address spaces) @@ expected void const volatile [noderef] __user *ptr @@ got char const *__gu_addr @@
expected void const volatile [noderef] __user *ptr
got char const *__gu_addr
incorrect type in initializer (different address spaces) @@ expected signed int const *__gu_addr @@ got signed int [noderef] [usertype] __user * @@
expected signed int const *__gu_addr
got signed int [noderef] [usertype] __user *
sparse: incorrect type in argument 1 (different address spaces) @@ expected void const volatile [noderef] __user *ptr @@ got signed int const *__gu_addr @@
expected void const volatile [noderef] __user *ptr
got signed int const *__gu_addr
context imbalance in 'vt_console_print' - wrong count at exit
sparse: incorrect type in initializer (different address spaces) @@ expected unsigned int const *__gu_addr @@ got unsigned int [noderef] __user * @@
expected unsigned int const *__gu_addr
got unsigned int [noderef] __user *
sparse: sparse: incorrect type in argument 1 (different address spaces) @@ expected void const volatile [noderef] __user *ptr @@ got unsigned int const *__gu_addr @@
expected void const volatile [noderef] __user *ptr
got unsigned int const *__gu_addr
sparse: incorrect type in initializer (different address spaces) @@ expected unsigned int const *__gu_addr @@ got unsigned int [noderef] __user * @@
expected unsigned int const *__gu_addr
got unsigned int [noderef] __user *
sparse: incorrect type in argument 1 (different address spaces) @@ expected void const volatile [noderef] __user *ptr @@ got unsigned int const *__gu_addr @@
expected void const volatile [noderef] __user *ptr
got unsigned int const *__gu_addr
b/scm/linux/kernel/git/torvalds/linux.git/commit/?id=e5fc436f06eef54ef512ea55a9db8eb9f2e76959">https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit/?id=e5fc436f06eef54ef512ea55a9db8eb9f2e76959
//git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git">https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git
9
2020-06-15 821
2020-06-15 822 /*
2020-06-15 823 * We handle the console-specific ioctl's here. We allow the
2020-06-15 824 * capability to modify any console, not just the fg_console.
2020-06-15 825 */
2020-06-15 826 int vt_ioctl(struct tty_struct *tty,
2020-06-15 827 unsigned int cmd, unsigned long arg)
2020-06-15 828 {
2020-06-15 829 struct vc_data *vc = tty->driver_data;
2020-06-15 830 void __user *up = (void __user *)arg;
2020-06-15 831 int i, perm;
2020-06-15 832 int ret;
2020-06-15 833
2020-06-15 834 /*
2020-06-15 835 * To have permissions to do most of the vt ioctls, we either have
2020-06-15 836 * to be the owner of the tty, or have CAP_SYS_TTY_CONFIG.
2020-06-15 837 */
2020-06-15 838 perm = 0;
2020-06-15 839 if (current->signal->tty == tty || capable(CAP_SYS_TTY_CONFIG))
2020-06-15 840 perm = 1;
2020-06-15 841
2020-06-15 842 ret = vt_k_ioctl(tty, cmd, arg, perm);
2020-06-15 843 if (ret != -ENOIOCTLCMD)
2020-06-15 844 return ret;
2020-06-15 845
2020-06-15 846 ret = vt_io_ioctl(vc, cmd, up, perm);
2020-06-15 847 if (ret != -ENOIOCTLCMD)
2020-06-15 848 return ret;
2020-06-15 849
2020-06-15 850 switch (cmd) {
2020-06-15 851 case TIOCLINUX:
2020-06-15 852 return tioclinux(tty, arg);
2005-04-16 853 case VT_SETMODE:
2005-04-16 854 {
2005-04-16 855 struct vt_mode tmp;
2005-04-16 856
2005-04-16 857 if (!perm)
2012-03-02 858 return -EPERM;
2020-06-15 859 if (copy_from_user(&tmp, up, sizeof(struct vt_mode)))
2020-06-15 860 return -EFAULT;
2020-06-15 861 if (tmp.mode != VT_AUTO && tmp.mode != VT_PROCESS)
2020-06-15 862 return -EINVAL;
2020-06-15 863
2011-01-25 864 console_lock();
2005-04-16 865 vc->vt_mode = tmp;
2005-04-16 866 /* the frsig is ignored, so we set it to 0 */
2005-04-16 867 vc->vt_mode.frsig = 0;
2007-02-10 868 put_pid(vc->vt_pid);
2007-02-10 869 vc->vt_pid = get_pid(task_pid(current));
2005-04-16 870 /* no switch is required -- saw@xxxxxxxxxxxx */
2005-04-16 871 vc->vt_newvt = -1;
2011-01-25 872 console_unlock();
2008-04-30 873 break;
2005-04-16 874 }
2005-04-16 875
2005-04-16 876 case VT_GETMODE:
2005-04-16 877 {
2005-04-16 878 struct vt_mode tmp;
2005-04-16 879 int rc;
2005-04-16 880
2011-01-25 881 console_lock();
2005-04-16 882 memcpy(&tmp, &vc->vt_mode, sizeof(struct vt_mode));
2011-01-25 883 console_unlock();
2005-04-16 884
2005-04-16 885 rc = copy_to_user(up, &tmp, sizeof(struct vt_mode));
2008-04-30 886 if (rc)
2020-06-15 887 return -EFAULT;
2008-04-30 888 break;
2005-04-16 889 }
2005-04-16 890
2005-04-16 891 /*
2005-04-16 892 * Returns global vt state. Note that VT 0 is always open, since
2005-04-16 893 * it's an alias for the current VT, and people can't use it here.
2005-04-16 894 * We cannot return state for more than 16 VTs, since v_state is short.
2005-04-16 895 */
2005-04-16 896 case VT_GETSTATE:
2005-04-16 897 {
2005-04-16 898 struct vt_stat __user *vtstat = up;
2005-04-16 899 unsigned short state, mask;
2005-04-16 900
2005-04-16 901 if (put_user(fg_console + 1, &vtstat->v_active))
2020-06-15 902 return -EFAULT;
2020-06-15 903
2005-04-16 904 state = 1; /* /dev/tty0 is always open */
2020-03-21 905 console_lock(); /* required by vt_in_use() */
2008-04-30 906 for (i = 0, mask = 2; i < MAX_NR_CONSOLES && mask;
2008-04-30 907 ++i, mask <<= 1)
2020-02-19 908 if (vt_in_use(i))
2005-04-16 909 state |= mask;
2020-03-21 910 console_unlock();
2020-06-15 911 return put_user(state, &vtstat->v_state);
2005-04-16 912 }
2005-04-16 913
2005-04-16 914 /*
2005-04-16 915 * Returns the first available (non-opened) console.
2005-04-16 916 */
2005-04-16 917 case VT_OPENQRY:
2020-03-21 918 console_lock(); /* required by vt_in_use() */
2005-04-16 919 for (i = 0; i < MAX_NR_CONSOLES; ++i)
2020-02-19 920 if (!vt_in_use(i))
2005-04-16 921 break;
2020-03-21 922 console_unlock();
2020-06-15 923 i = i < MAX_NR_CONSOLES ? (i+1) : -1;
2020-06-15 924 return put_user(i, (int __user *)arg);
2005-04-16 925
2005-04-16 926 /*
2005-04-16 927 * ioctl(fd, VT_ACTIVATE, num) will cause us to switch to vt # num,
2005-04-16 928 * with num >= 1 (switches to vt 0, our console, are not allowed, just
2005-04-16 929 * to preserve sanity).
2005-04-16 930 */
2005-04-16 931 case VT_ACTIVATE:
2005-04-16 932 if (!perm)
2012-03-02 933 return -EPERM;
2005-04-16 934 if (arg == 0 || arg > MAX_NR_CONSOLES)
2020-06-15 935 return -ENXIO;
2020-06-15 936
2005-04-16 937 arg--;
2011-01-25 938 console_lock();
2008-04-30 939 ret = vc_allocate(arg);
2011-01-25 940 console_unlock();
2008-04-30 941 if (ret)
2020-06-15 942 return ret;
2005-04-16 943 set_console(arg);
2008-04-30 944 break;
2005-04-16 945
2009-09-19 946 case VT_SETACTIVATE:
2009-09-19 947 if (!perm)
2012-03-02 948 return -EPERM;
2009-09-19 949
2020-06-15 950 return vt_setactivate(up);
2009-09-19 951
2005-04-16 952 /*
2005-04-16 953 * wait until the specified VT has been activated
2005-04-16 954 */
2005-04-16 955 case VT_WAITACTIVE:
2005-04-16 956 if (!perm)
2012-03-02 957 return -EPERM;
2005-04-16 958 if (arg == 0 || arg > MAX_NR_CONSOLES)
2020-06-15 959 return -ENXIO;
2020-06-15 960 return vt_waitactive(arg);
2005-04-16 961
2005-04-16 962 /*
2005-04-16 963 * If a vt is under process control, the kernel will not switch to it
2005-04-16 964 * immediately, but postpone the operation until the process calls this
2005-04-16 965 * ioctl, allowing the switch to complete.
2005-04-16 966 *
2005-04-16 967 * According to the X sources this is the behavior:
2005-04-16 968 * 0: pending switch-from not OK
2005-04-16 969 * 1: pending switch-from OK
2005-04-16 970 * 2: completed switch-to OK
2005-04-16 971 */
2005-04-16 972 case VT_RELDISP:
2005-04-16 973 if (!perm)
2012-03-02 974 return -EPERM;
2005-04-16 975
2012-03-02 976 console_lock();
2020-06-15 977 ret = vt_reldisp(vc, arg);
2012-03-02 978 console_unlock();
2005-04-16 979
2020-06-15 980 return ret;
2020-06-15 981
2005-04-16 982
2005-04-16 983 /*
2005-04-16 984 * Disallocate memory associated to VT (but leave VT1)
2005-04-16 985 */
2005-04-16 986 case VT_DISALLOCATE:
2020-06-15 987 if (arg > MAX_NR_CONSOLES)
2020-06-15 988 return -ENXIO;
2020-06-15 989
2013-05-17 990 if (arg == 0)
2013-05-17 991 vt_disallocate_all();
2013-05-17 992 else
2020-06-15 993 return vt_disallocate(--arg);
2008-04-30 994 break;
2005-04-16 995
2005-04-16 996 case VT_RESIZE:
2005-04-16 997 {
2005-04-16 998 struct vt_sizes __user *vtsizes = up;
A. Daplas 2007-10-16 999 struct vc_data *vc;
2005-04-16 1000 ushort ll,cc;
2020-06-15 1001
2005-04-16 1002 if (!perm)
2012-03-02 1003 return -EPERM;
2005-04-16 @1004 if (get_user(ll, &vtsizes->v_rows) ||
2005-04-16 1005 get_user(cc, &vtsizes->v_cols))
2020-06-15 1006 return -EFAULT;
2020-06-15 1007
2011-01-25 1008 console_lock();
A. Daplas 2007-10-16 1009 for (i = 0; i < MAX_NR_CONSOLES; i++) {
A. Daplas 2007-10-16 1010 vc = vc_cons[i].d;
A. Daplas 2007-10-16 1011
A. Daplas 2007-10-16 1012 if (vc) {
A. Daplas 2007-10-16 1013 vc->vc_resize_user = 1;
2012-03-02 1014 /* FIXME: review v tty lock */
2008-08-15 1015 vc_resize(vc_cons[i].d, cc, ll);
A. Daplas 2007-10-16 1016 }
A. Daplas 2007-10-16 1017 }
2011-01-25 1018 console_unlock();
2008-04-30 1019 break;
2005-04-16 1020 }
2005-04-16 1021
2005-04-16 1022 case VT_RESIZEX:
2005-04-16 1023 if (!perm)
2012-03-02 1024 return -EPERM;
2020-02-10 1025
2020-06-15 1026 return vt_resizex(vc, up);
2005-04-16 1027
2005-04-16 1028 case VT_LOCKSWITCH:
2005-04-16 1029 if (!capable(CAP_SYS_TTY_CONFIG))
2012-03-02 1030 return -EPERM;
2020-02-19 1031 vt_dont_switch = true;
2008-04-30 1032 break;
2005-04-16 1033 case VT_UNLOCKSWITCH:
2005-04-16 1034 if (!capable(CAP_SYS_TTY_CONFIG))
2012-03-02 1035 return -EPERM;
2020-02-19 1036 vt_dont_switch = false;
2008-04-30 1037 break;
2006-08-27 1038 case VT_GETHIFONTMASK:
2020-06-15 1039 return put_user(vc->vc_hi_font_mask,
2008-04-30 1040 (unsigned short __user *)arg);
2009-09-19 1041 case VT_WAITEVENT:
2020-06-15 1042 return vt_event_wait_ioctl((struct vt_event __user *)arg);
2005-04-16 1043 default:
2020-06-15 1044 return -ENOIOCTLCMD;
2005-04-16 1045 }
2020-06-15 1046
2020-06-15 1047 return 0;
2005-04-16 1048 }
2005-04-16 1049
by commit
Linux-2.6.12-rc2
xx>
xx>
itty/list/kbuild-all@xxxxxxxxxxxx">https://lists.01.org/hyperkitty/list/kbuild-all@xxxxxxxxxxxx
/strong>
- Next message: Randy Dunlap: "Re: xconfig is broken again on Qt5"
- Previous message: Yong Wu: "Re: [PATCH v2 01/23] dt-bindings: iommu: mediatek: Convert IOMMU to DT schema"
- Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]