notify.c: space checkpatch "fixes"

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

kdbus.h: fix stupid checkpatch warnings about spaces in comments.

ugh, trivial crud...

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

fix semicolon in if statement

Signed-off-by: Govindarajulu Varadarajan <govindarajulu90@gmail.com>

Fix some stupid checkpatch warnings in connection.h and names.h

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

limit the amount of request for a reply a connection can issue

test: benchmark - use sizeof(struct timeval)

Simon Peeters 12:41 PM:
I am happy to let you all know that the error is not in the kdbus
code, but in the test code which has 16 hardcoded as the size of struct
timeval.

test: cast addr to uintptr_t

grant CAP_IPC_OWNER-like access to the owner of the bus

do not accept "kernel messages" from userspace

kdbus.h: fix kernel doc

kdbus.h: KDBUS_ITEM_PRIORITY is not implemented and probably not even a good idea, let's remove it

use module name for printk

print better error message

TODO: update

memfd: use KBUILD_MODNAME

allow to compile a separate kernel module with a named extension

running:
  make EXT=2

will create a:
  kdbus2.ko

and the tests will run against:
  /dev/kdbus2/control

This allows us to run a new and separated kernel modules while the original
kdbus.ko is in use in the booted system.

fix kernel doc

handle timeouts only handled for expected replies, not for queued messages

TODO: update

connection: fix access-after-free

update cookie and cookie_reply logic

return error in kdbus_name_release()

do not deny writing messages to an activator connection

handle: do not put "ep" and "conn" in a union

metadata: always allocate and don't embed meta object

connection: move messages addressed to a name back to the activator

metadata: use append_data for credentials

metadata: re-initialize meta object, we might re-use it

allow HELLO to impersonate somebody else

match: check for currently owned names

policy: remove inline declaration, it's the compiler's decision

connection: do not guess-check for target connection without ref'ing it

rename "id" to "owner_id" for all actions on behalf of other connections

TODO: update

do not filter directed messages sent to monitor connections

names: cosmetics

names: fix 'added' assignment

names: rework list logic once more

now, KDBUS_NAME_LIST_ACTIVATORS will list name entries regardless
whether they have been activated or not.

kdbus.h: document ENOENT error

names: fix condition for activator listing

names: add the activator connection itself to the list of of activator names

add KDBUS_ITEM_DST_NAME to outgoing messages

metadata: set every bit only once, also bits for unknown attach requests

match: make kdbus_match_entry_free static

notify: assign kmsg's notify_{new,old}_id

change error message of BYEBYE to -EBUSY

test-kdbus: add tests for match logic

match: rework code

This is a rework of the match code logic. This also changes the external
API in multiple ways. In particular,

 * the item types of kdbus_cmd_match are now of type KDBUS_ITEM_*
 * existing kdbus_item structs are used as attachment to kdbus_cmd_match
 * the KDBUS_MATCH definitions have been removed entirely
 * introduce KDBUS_ITEM_ID to store a connection ID
 * src_id in kdbus_cmd_match has been removed, the information lives in
   an item attachment now.

Internally, the code was completely reworked.

kdbus-test: start monitor tests

match: add name matching, match kernel notifications

match: read the name from the right structure

do not allow monitor connections to own names

connection: make KDBUS_ITEM_PAYLOAD_VEC's offset relative to the message start

This way, we don't have to modify the content of such items when copying
around messages from one pool to another, as done when activator names
are taken over.

notify: always use memcpy

notify: make sure we send NUL terminate name strings in name_change messages to userspace

test: chat - dump all names

names: use the correct list entry when iterating over queued names

names: remember entry when queueing up a connection for a name

names: clean up flag handling when taking over names

names: set incoming/outgoing queueing flags in one place

names: directly pass KDBUS_NAME_ACTIVATOR for activator requests

names: strict filter of incoming KDBUS_NAME_* flags

do not return improper idr failure codes to userspace

endpoint: fix idr_alloc return value check

Revert "move kdbus_subsys from namespace.c to main.c"

This reverts commit 1ca27c6e7e95d505f6bee190b8dd30dea6f415d1.

match: plug memory leak

bus: don't try to free an ERR_PTR

bus: fix memory leak on malformed messages

endpoint: fix memory leak on malformed input

endpoint: save an extra variable for idr_alloc()

endpoint: plug mem leak if kstrdup() fails

namespace: don't try to free an ERR_PTR

indent

move kdbus_subsys from namespace.c to main.c

pool: fix include

indent

fix multi-line comment

kernel-doc kdbus_handle_type

more include cleanups

Makefile: add install/uninstall

kdbus.h: fix kernel-doc

make kdbus_notify_name_change consist of two kdbus_notify_id_change

kdbus.h cosmetics

handle: clean up error path

no functional change.

include linux/uaccess.h from all files that use copy_{from,to}_user

include uaccess.h where needed

test-kdbus: add test for BYEBYE

document KDBUS_CMD_BYEBYE

add KDBUS_CMD_BYEBYE

handle: plug memory leak in case copy_to_user() fails

pool: use {} in all conditional branches if used in one of them

avoid block in case statements unless necessary

remove kdbus_msg_dump

connection: use kcalloc when allocating arrays

remove unneeded include headers

headers: more trivial style complaints

some trivial style-related fixups

namespace: pre-calculate payload_size to simplify code

kdbus_msg_scan_items(): simplify item size logic

namespace: catch return var of register_chrdev() in signed int