Fixed memory freeing in match.c

Change-Id: If3b7df12b8ded42ca7e98788482ad3369166e379
Signed-off-by: Radoslaw Pajak <r.pajak@samsung.com>

[fix] fixed memory freeing on errors

Change-Id: Iea8693d253c3a325b769d2454db0029b9a6e48a8
Signed-off-by: Radoslaw Pajak <r.pajak@samsung.com>

[fix] Request name returns information if connection is already owner of the name

Request name fixed to return errno = -EALREADY if connection wants to own well-known name, that he is owner already.
This is needed to return DBUS_DBUS_REQUEST_NAME_REPLY_ALREADY_OWNER, which was not possible before.

Fixed memory freeing in kdbus_cmd_name_acquire function.

Change-Id: I6dca3d53e22d115f3d366323dee80a2ea771495f
Signed-off-by: Radoslaw Pajak <r.pajak@samsung.com>

[feature-fix] from now starters are not appended to KDBUS_MSG_SRC_NAMES

Change-Id: I39c1cf1284956c3a6508a516d076bbb6b9a97dc4
Signed-off-by: Radoslaw Pajak <r.pajak@samsung.com>

Use current_euid and current_egid for user credentials

Copyright (C) 2013  Samsung Electronics added in appropriate files

Change-Id: Ibd6f68066a297dbd19e4f8e534cb28c5806c1bc9
Signed-off-by: Radoslaw Pajak <r.pajak@samsung.com>

Merge branch 'open' of ssh://mcdsrvbld02.digital.local/home/DIGITAL/d.kowalski/kdbus-open into open

Copy mode to endpoint struct in kdbus_ep_new()

Authored-by: HyungJun Choi <hj210.choi@samsung.com>
Signed-off-by: Daniel Kowalski <d.kowalski@samsung.com>

Merge branch 'kernel-3.8.2' into open

Fixed releasing name when being in the queue for that name.

Change-Id: Idb9f828085e922e8e9330a8c4d967daa82a60ad8
Signed-off-by: Radoslaw Pajak <r.pajak@samsung.com>

From now starters are not listed by kdbus_cmd_name_query

Change-Id: I9cbe75cc1ef5db6512244b405c38549907511c61
Signed-off-by: Radoslaw Pajak <r.pajak@samsung.com>

Fixed name releasing and listing

- Fixed names listing - from now starters are not listed
- Fixed name releasing when starter becomes owner of the name
- Fixed broadcasting name changes - from now starters are hidden - they are not presented as owners of the names

Change-Id: I34aec7f1a52a754f2583cd88ef68fa83006337e8
Signed-off-by: Radoslaw Pajak <r.pajak@samsung.com>

[feature-fix] fix for registering starters as names

- fix for added feature, that connection can register name with KDBUS_NAME_STARTER flag, which could
 lead to daemon crash and ignore kill

Change-Id: Iccecc561dda1bb1d4d8af38196a2e11eb86812a8
Signed-off-by: Radoslaw Pajak <r.pajak@samsung.com>

well-known name can be registered as a starter

Now application can request the well-known name with starter flag (KDBUS_NAME_STARTER)
with the same functionality as KDBUS_HELLO_STARTER flag give to the connection

Signed-off-by: Radoslaw Pajak <r.pajak@samsung.com>
Change-Id: I7df87911aa5d55b8f2116c5d814e71a5002ee739

Conflicts:

names.c

Compatibility: prototypes moved to wrapper headers

Avoided including compatibility.h except message
cleanup of test-kdbus.c

Change-Id: I07b9909b648b69e755e3e49fba818da9adb18cf9

Conflicts:

Makefile

Compatibility: small fix for 3.10.x kernels

Change-Id: I3579655775df8d5e4d62b859fe762fd04babd79a

Compatibility: Michal's fixes to compatibility package

Updated conditionals.

Change-Id: I83db53309be95df0a397b750ea9feecdc79fa179

support for 3.8 kernel added in Makefile and c files

compatibility header is used.
kernel patches added
UPDATE all patches merged into one
UPDATE missing kernel functions moved to compatibility.c

Change-Id: Ia6271423e985bcda32f9f566edf5431b2888ce5e

Initial commit with compatibility package for 3.8 kernel

basic macros and 2 missing API functions

Change-Id: Iaba27c9b898c522faf94bbc8b885714cc9ee0686

use "&&" instead of nested one-line "if"

fixed name queueing when handling conflicts

Connection that loses name is put into queue if it wish to.

Signed-off-by: Radoslaw Pajak <r.pajak@samsung.com>

Fixed name releasing and sending broadcasts about name change

- fixed name releasing, when starter acquires the name released
- fixed old_id when sending KDBUS_MSG_NAME_CHANGE after owner has changed
- fixed condition to handle name conflict if there is starter in background
- fixed sending KDBUS_MSG_NAME_ADD only when the name wasn't present on the bus

Signed-off-by: Radoslaw Pajak <r.pajak@samsung.com>

Release name fixed

Release name fixed to allow name releasing by priviliged user on behalf of other peer, if that peer own the name.

Signed-off-by: Radoslaw Pajak <r.pajak@samsung.com>

Fixed adding and removing match rules for other peers

Signed-off-by: Radoslaw Pajak <r.pajak@samsung.com>

match: move nested loop to separate function

Fixed broadcasts and matching

- fixed kernel notify messages of changes of ids and names on the bus
- message matching fixed for user broadcasts
- message matching fixed for kernel broadcasts

Signed-off-by: Radoslaw Pajak <r.pajak@samsung.com>

allocate only the alignment bytes for \0-bytes records

Reported-by: Daniel Kowalski <d.kowalski@samsung.com>

Merge branch 'master' of https://code.google.com/p/d-bus

preserve payload record padding when \0-bytes records are used

Based on a patch from: Daniel Kowalski <d.kowalski@samsung.com>

namespace: hold mutex while creating a new instance

some minor fixes for endpoint and namespace

 endpoint.c: fix name comparison for endpoint
namespace.c: don't try to find when name is NULL

Signed-off-by: Przemyslaw Kedzierski <p.kedzierski@samsung.com>

Merge pull request #4 from artisdom/patch-1

kdbus.txt: typo fix.

kdbus.txt: typo fixes

Some corrections to make test-kdbus work on i686.

Signed-off-by: Przemyslaw Kedzierski <p.kedzierski@samsung.com>

Merge pull request #3 from phomes/master

kdbus.txt: typo fixes

kdbus.txt: typo fixes

kdbus: fix some memory leaks

These were found by cppcheck.

Signed-off-by: Tero Roponen <tero.roponen@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>

catch up with final working version of cgroups string interface

use hierarchy == 2 for now

test/kdbus-util.c: kill stray ')'

pool: plug mem leak in error unwind of kdbus_pool_init

do not expose cgroup hierarchy in the API, assume *the* hierarchy

The returned cgroup path is always "/" until Tejun's fix hit the
current kernel, it's pending ...

message.c: task_cgroup_path_from_hierarchy is now in 3.11-rc1, so remove it from here

Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>

Merge pull request #2 from Keruspe/master

Fix build with CONFIG_UIDGID_STRICT_TYPE_CHECKS=y

connection: fix build with CONFIG_UIDGID_STRICT_TYPE_CHECKS=y

connection.c: In function ‘kdbus_conn_ioctl_control’:
connection.c:1047:8: error: incompatible types when assigning to type ‘gid_t’ from type ‘kgid_t’
connection.c:1051:9: error: incompatible type for argument 5 of ‘kdbus_bus_new’

Signed-off-by: Marc-Antoine Perennou <Marc-Antoine@Perennou.com>

bus: fix build with CONFIG_UIDGID_STRICT_TYPE_CHECKS=y

bus.c: In function ‘kdbus_bus_uid_is_privileged’:
bus.c:37:21: error: invalid operands to binary == (have ‘kuid_t’ and ‘kuid_t’)

Signed-off-by: Marc-Antoine Perennou <Marc-Antoine@Perennou.com>

test-kdbus.c: get it to work on 32bit systems.

Belisko Marek <marek.belisko@gmail.com> reports that without aligning
the bus_make variable on an 8byte boundry, as happens on 32bit systems,
the test fails.  This should resolve this issue.

Merge pull request #1 from vbalyasnyy/master

cleaning of security label

connection.c: use security_release_secctx to free security label

pool.c: tiny checkpatch.pl cleanup

test: use 1024 for a kB :)

prefix all functions with kdbus_

mark kdbus_conn_timeout_schedule_scan as static

fix up missing __user marking for kdbus_pool_write_user()

Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>

KDBUS_PTR is really a __user pointer, so mark it as such.

return proper -EINVAL instead of EINVAL for an error

external API: use explicit padding

Data structures with 64 bit members in them differ in alignment
by architecture: x86-32 keeps the alignment at 32 bit at most,
while all sane architectures including x86-64 use 64 bit
alignment and add extra implicit padding if necessary.

This means the kdbus_cmd_monitor structure is incompatible between
x86-64 and x86-32, unless we make the padding explcicit.

struct kdbus_memfd has a similar definition, so we change that
as well, although the change currently makes no difference because
it is always embedded in a larger structure.

Signed-off-by: Arnd Bergmann <arnd@arndb.de>

external API: fix ioctl command definitions

This changes the ioctl commands to follow the common
conventions, resolving one item from the TODO file:

* We use _IO for commands that use the argument as a literal
  and not dereference it as a pointer. KDBUS_CMD_MEMFD_SEAL_SET
  just passes an bool here, and KDBUS_CMD_MSG_RELEASE passes
  a pointer that is compared to other pointers but not accessed.

* _IOR is used for commands that only copy data from the kernel
  to user space, IOW is used for commands that only copy into
  the kernel, which is the majority here. _IOWR is used only
  for commands that both read and write on the pointer passed
  as the ioctl arg. Whether we read or write from indirect
  pointers passed in the structure is not relevant here.

* The size argument must be the data structure that the ioctl
  argument points to. Importantly, this is not a pointer type
  as e.g. in the old KDBUS_CMD_MSG_RECV definition. Using
  a pointer here breaks 32 bit compat mode because taking
  sizeof(__u64*) results in a different command number on 32 and
  64 bit user space.

An open question is what to do for commands with variable-length
arguments. Some subsystems in the kernel actually define variable-length
commands as

 #define KDBUS_CMD_MSG_SEND(len) _IOC(_IOC_WRITE,KDBUS_IOC_MAGIC,\
 0x40,sizeof(struct kdbus_msg)+(len))

This has the advantage of saving one 'copy_from_user' for each
call, and actually reflecting the correct size in the command
number, but it can also confuse readers. The maximum size that
can be passed like this is 8192 (alpha, mips, sparc, powerpc)
or 16384 (all other architectures) bytes, which may also be a
limitation.

Signed-off-by: Arnd Bergmann <arnd@arndb.de>

pool: use shmem file as backing store

use subsys_virtual_register()

bus: remove connection list, use the hash list

TODO: update

make part iteration generic; *really* validate parts, fix now failing test

minor coding style cleanups to keep checkpatch happy

Merge branch 'master' of github.com:gregkh/kdbus

test: names - do not iterate two items at once

namespace: fix up another -EINVAL error

return proper -EINVAL instead of EINVAL for an error

kdbus_size_set_user() had the arguments backwards in copy_to_user()

names: add KDBUS_ITEM_NEXT, pad to alignment

names: use KDBUS_ITEM_NEXT to pad to alignment; avoid sizeof(item)

connection: link into bus only after HELLO, not at open()

pool: accept only anonymous memory

track connection state in 'type' not in 'active'

kdbus: prefix NULL enums with an underscore since they aren't really supposed to be used

allow privileged users to act on behalf of somebody else

track connection in the bus instead of the endpoint; add monitoring

match: swap filter and mask

match: fix !

match: accept cmd_match->src_id == 0 for "self"

ennforce data alignment in the API where neccessary

do not rely on 64bit put_user()

add missing cast to kdbus_size_set_user()

make sure we always pass __u64 in the API, never long or pointer

use common IS_ALIGNED() for PAGE_SIZE check

move sending code from message.c to connection.c

pool: do not hand out more than half of the pool size with a single request

pool: name page variables with common prefix

Merge branch 'master' of https://code.google.com/p/d-bus

pool: kdbus_pool_cleanup() - allow a NULL pool

fix up sparse warnings.

memfd: fix storage size for pointers

memfd: mmap() - grow file only if really needed

dump() at info level

do not mangle temporary list entry

always copy into the *mapped* slice in the receiver's pool

pool: implement allocator

make struct kdbus_conn_queue private to connection.c

remove size limit for "padding vecs"

support vec.address == NULL; vec.size = 1..7; as a "padding vec"