[kbuild] [char-misc:kdbus 12/13] ipc/kdbus/handle.c:736 handle_ep_ioctl_connected() warn: maybe return -EFAULT instead of the bytes remaining?

2014-11-24 Thread Dan Carpenter
tree:   git://git.kernel.org/pub/scm/linux/kernel/git/gregkh/char-misc.git kdbus
head:   d064b29b452333d0e0ed6b9d2fc96b6e3917b1d7
commit: 0311aa1370e08cf0bf799a5f6e7ea29297cc3bdd [12/13] kdbus: add Makefile, 
Kconfig and MAINTAINERS entry

ipc/kdbus/handle.c:736 handle_ep_ioctl_connected() warn: maybe return -EFAULT 
instead of the bytes remaining?
ipc/kdbus/metadata.c:376 kdbus_meta_append_cmdline() warn: maybe return -EFAULT 
instead of the bytes remaining?

git remote add char-misc 
git://git.kernel.org/pub/scm/linux/kernel/git/gregkh/char-misc.git
git remote update char-misc
git checkout 0311aa1370e08cf0bf799a5f6e7ea29297cc3bdd
vim +736 ipc/kdbus/handle.c

97149087 Daniel Mack 2014-09-11  720ret = 
kdbus_negotiate_flags(cmd_free, buf, typeof(cmd_free),
97149087 Daniel Mack 2014-09-11  7210);
97149087 Daniel Mack 2014-09-11  722if (ret  0)
97149087 Daniel Mack 2014-09-11  723break;
97149087 Daniel Mack 2014-09-11  724  
97149087 Daniel Mack 2014-09-11  725ret = 
kdbus_pool_release_offset(conn-pool, cmd_free.offset);
97149087 Daniel Mack 2014-09-11  726break;
97149087 Daniel Mack 2014-09-11  727}
97149087 Daniel Mack 2014-09-11  728  
97149087 Daniel Mack 2014-09-11  729default:
97149087 Daniel Mack 2014-09-11  730ret = -ENOTTY;
97149087 Daniel Mack 2014-09-11  731break;
97149087 Daniel Mack 2014-09-11  732}
97149087 Daniel Mack 2014-09-11  733  
97149087 Daniel Mack 2014-09-11  734kdbus_conn_release(conn);
97149087 Daniel Mack 2014-09-11  735kfree(free_ptr);
97149087 Daniel Mack 2014-09-11 @736return ret;
97149087 Daniel Mack 2014-09-11  737  }
97149087 Daniel Mack 2014-09-11  738  
97149087 Daniel Mack 2014-09-11  739  /* kdbus endpoint commands for endpoint 
owners */
97149087 Daniel Mack 2014-09-11  740  static long handle_ep_ioctl_owner(struct 
file *file, unsigned int cmd,
97149087 Daniel Mack 2014-09-11  741  void __user 
*buf)
97149087 Daniel Mack 2014-09-11  742  {
97149087 Daniel Mack 2014-09-11  743struct kdbus_handle_ep *handle = 
file-private_data;
97149087 Daniel Mack 2014-09-11  744struct kdbus_ep *ep = handle-ep_owner;

---
0-DAY kernel test infrastructureOpen Source Technology Center
http://lists.01.org/mailman/listinfo/kbuild Intel Corporation
___
kbuild mailing list
kbuild@lists.01.org
https://lists.01.org/mailman/listinfo/kbuild


[kbuild] [char-misc:kdbus 12/13] ipc/kdbus/handle.c:736 handle_ep_ioctl_connected() warn: maybe return -EFAULT instead of the bytes remaining?

2014-11-20 Thread kbuild test robot
TO: Daniel Mack dan...@zonque.org
CC: Greg Kroah-Hartman gre...@linuxfoundation.org
CC: David Herrmann dh.herrm...@gmail.com
CC: Djalal Harouni tix...@opendz.org

tree:   git://git.kernel.org/pub/scm/linux/kernel/git/gregkh/char-misc.git kdbus
head:   d064b29b452333d0e0ed6b9d2fc96b6e3917b1d7
commit: 0311aa1370e08cf0bf799a5f6e7ea29297cc3bdd [12/13] kdbus: add Makefile, 
Kconfig and MAINTAINERS entry
:: branch date: 78 minutes ago
:: commit date: 78 minutes ago

ipc/kdbus/handle.c:736 handle_ep_ioctl_connected() warn: maybe return -EFAULT 
instead of the bytes remaining?
ipc/kdbus/metadata.c:376 kdbus_meta_append_cmdline() warn: maybe return -EFAULT 
instead of the bytes remaining?

git remote add char-misc 
git://git.kernel.org/pub/scm/linux/kernel/git/gregkh/char-misc.git
git remote update char-misc
git checkout 0311aa1370e08cf0bf799a5f6e7ea29297cc3bdd
vim +736 ipc/kdbus/handle.c

97149087 Daniel Mack 2014-09-11  720ret = 
kdbus_negotiate_flags(cmd_free, buf, typeof(cmd_free),
97149087 Daniel Mack 2014-09-11  7210);
97149087 Daniel Mack 2014-09-11  722if (ret  0)
97149087 Daniel Mack 2014-09-11  723break;
97149087 Daniel Mack 2014-09-11  724  
97149087 Daniel Mack 2014-09-11  725ret = 
kdbus_pool_release_offset(conn-pool, cmd_free.offset);
97149087 Daniel Mack 2014-09-11  726break;
97149087 Daniel Mack 2014-09-11  727}
97149087 Daniel Mack 2014-09-11  728  
97149087 Daniel Mack 2014-09-11  729default:
97149087 Daniel Mack 2014-09-11  730ret = -ENOTTY;
97149087 Daniel Mack 2014-09-11  731break;
97149087 Daniel Mack 2014-09-11  732}
97149087 Daniel Mack 2014-09-11  733  
97149087 Daniel Mack 2014-09-11  734kdbus_conn_release(conn);
97149087 Daniel Mack 2014-09-11  735kfree(free_ptr);
97149087 Daniel Mack 2014-09-11 @736return ret;
97149087 Daniel Mack 2014-09-11  737  }
97149087 Daniel Mack 2014-09-11  738  
97149087 Daniel Mack 2014-09-11  739  /* kdbus endpoint commands for endpoint 
owners */
97149087 Daniel Mack 2014-09-11  740  static long handle_ep_ioctl_owner(struct 
file *file, unsigned int cmd,
97149087 Daniel Mack 2014-09-11  741  void __user 
*buf)
97149087 Daniel Mack 2014-09-11  742  {
97149087 Daniel Mack 2014-09-11  743struct kdbus_handle_ep *handle = 
file-private_data;
97149087 Daniel Mack 2014-09-11  744struct kdbus_ep *ep = handle-ep_owner;

:: The code at line 736 was first introduced by commit
:: 971490875e286b89cc101411ea275a9928cb5397 kdbus: add driver skeleton, 
ioctl entry points and utility functions

:: TO: Daniel Mack dan...@zonque.org
:: CC: Greg Kroah-Hartman gre...@linuxfoundation.org

---
0-DAY kernel test infrastructureOpen Source Technology Center
http://lists.01.org/mailman/listinfo/kbuild Intel Corporation
___
kbuild mailing list
kbuild@lists.01.org
https://lists.01.org/mailman/listinfo/kbuild