kdbus: fix g_dbus_connection_call_sync ("NameHasOwner") failure

The caller _dbus_daemon_synthetic_reply function uses the error for a special purpose.
So, _g_kdbus_NamehasOwner never makes error and only return FALSE even if an error occurs.

================ the call stck of failure =====================================
0  g_kdbus_NameHasOwner_internal (worker=worker@entry=0x42a40, name=name@entry=0x5df78 "org.not.existing.busname") at gkdbus.c:1190
1  0xb6f18a0c in _g_kdbus_NameHasOwner (worker=worker@entry=0x42a40, name=0x5df78 "org.not.existing.busname", error=error@entry=0xbefff6d0) at gkdbus.c:1314   ==> generate gdbus error
2  0xb6f1a526 in _dbus_daemon_synthetic_reply (worker=worker@entry=0x42a40, message=message@entry=0x57c60) at gkdbusfakedaemon.c:542  ==> This function only check error value rather than return value.  So, after appling patch, this logic has been changed.
3  0xb6f19a9c in _g_kdbus_worker_send_message_sync (worker=0x42a40, message=message@entry=0x57c60, out_reply=out_reply@entry=0xbefff768, timeout_msec=timeout_msec@entry=-1, cancellable=cancellable@entry=0x0, error=error@entry=0xbefff7ac) at gkdbus.c:3888
4  0xb6f3fe78 in g_dbus_connection_send_message_with_reply_sync (connection=connection@entry=0x421e0, message=message@entry=0x57c60, flags=G_DBUS_SEND_MESSAGE_FLAGS_NONE, timeout_msec=timeout_msec@entry=-1, out_serial=0xbefff76c, out_serial@entry=0x0, cancellable=0x0, error=error@entry=0xbefff7ac) at gdbusconnection.c:3019
5  0xb6f4013e in g_dbus_connection_call_sync_internal (connection=0x421e0, bus_name=<optimized out>, object_path=0x24058 "/org/freedesktop/DBus", interface_name=0x24040 "org.freedesktop.DBus", method_name=method_name@entry=0x24030 "NameHasOwner", parameters=parameters@entry=0x3d6a8, reply_type=reply_type@entry=0x24130, flags=flags@entry=G_DBUS_CALL_FLAGS_NONE, timeout_msec=timeout_msec@entry=-1, fd_list=fd_list@entry=0x0, out_fd_list=out_fd_list@entry=0x0, cancellable=cancellable@entry=0x0, error=error@entry=0xbefff83c) at gdbusconnection.c:6641
6  0xb6f4187a in g_dbus_connection_call_sync (connection=<optimized out>, bus_name=<optimized out>, object_path=<optimized out>, interface_name=<optimized out>, method_name=0x24030 "NameHasOwner", parameters=0x3d6a8, reply_type=0x24130, flags=G_DBUS_CALL_FLAGS_NONE, timeout_msec=-1, cancellable=0x0, error=0xbefff83c) at gdbusconnection.c:6868
==============================================================================

Signed-off-by: INSUN PYO <insun.pyo@samsung.com>
Change-Id: I4959eaeeaad79de641f6b0c1ce10020ae9fe3760

Remove Profile Build Dependency

Apply Tizen-TV profile optimization for ALL Tizen profiles
because it would not have side effects on the correctness.
(simply ref/unref a Tizen-wide "dbus")

Change-Id: I209f9bf798b02f674a95dfc38142b4a44e194883
Signed-off-by: MyungJoo Ham <myungjoo.ham@samsung.com>

gdbus:add null check for error variable

Change-Id: Ib51dd62ebc066b9393ae7c59f2f63d3330a16518
Signed-off-by: sanghyeok.oh <sanghyeok.oh@samsung.com>
(cherry picked from commit 22aff71838c2aafc27196886fff317d146f749e1)

Kdbus: Fix for handling EPERM errors

Change-Id: I63b35469d0fe660a33880ac02b9dcb405ceeafac
Signed-off-by: Hyotaek Shim <hyotaek.shim@samsung.com>

kdbus: check return status of KDBUS_CMD_FREE iotcl

Signed-off-by: INSUN PYO <insun.pyo@samsung.com>
Change-Id: I35ae1afffc01aaf1df54d6d2466cb0ecb3417c73

kdbus: check return status of KDBUS_CMD_CONN_INFO iotcl

Change-Id: If6ea3c1825333aca2af4a2599b0187b1ff931b34

gdbus: change timeout error code from 'gdbus error' to 'gio error'

sync/async call return different error code when timeout was occured
modify it to return G_IO_ERROR_TIMED_OUT

Change-Id: I7b9fb847240311ce40f59ab7d552aaa2efeecb45
Signed-off-by: sanghyeok.oh <sanghyeok.oh@samsung.com>
(cherry picked from commit 7a6fcd470025060f220b51ed8a4ecd57cf2b84b7)

kdbus: fix race-condition in GDBusNameWatching component

To avoid race-condition between GetNameOwner replay and
NameOwnerChanged signal in GDBusNameWatching component,
let's always use GetNameOwner sync call on kdbus bus.

Change-Id: Iebb780737d753d989a7d928f65d433d86d79284a

coding style:errno must referenced right after calling standard function

Change-Id: I8eff8b174ecb621acbbe73734aa5c35cf5c60bfc
Signed-off-by: sanghyeok.oh <sanghyeok.oh@samsung.com>

Fix for missing g_source_unref(Gkdbus worker source)

Change-Id: Ia5fb338b8b69b76c52d2dee8f6ccbb39e101a1c7
Signed-off-by: Hyotaek Shim <hyotaek.shim@samsung.com>

glib:remove self-join deadlock:change to g_thread_unref

Change-Id: I1b98bed183510ca79c3448da5e2bd2a4115de709
Signed-off-by: sanghyeok.oh <sanghyeok.oh@samsung.com>

glib:remove self-join deadlock

Change-Id: Ia9fffe62c90d596efe29929c50e835d4a93fac52
Signed-off-by: sanghyeok.oh <sanghyeok.oh@samsung.com>

fix double unref

Allocated memory by g_thread_new() is freed by g_thread_join() or g_thread_unref()
It is not needed to use both g_thread_join() and g_thread_unref().

Change-Id: Ic6d13ecfa9f6c8d265bf516f4438e7a83bbd0e9c
Signed-off-by: sanghyeok.oh <sanghyeok.oh@samsung.com>

kdbus: Add a null teminated string for GetConnectionCredentials (LinuxSecurityLabel) method.

The result of LinuxSecurityLabel item should end to null.
Dbus spec : https://dbus.freedesktop.org/doc/dbus-specification.html

Legacy dbus daemon : dbus/bus/driver.c. _dbus_asv_add_byte_array (&array_iter, "LinuxSecurityLabel", s, strlen (s) + 1))
libdbus kdbus : dbus-transport-kdbus.c. dbus_bool_t res = _dbus_asv_add_byte_array (&array_iter, "LinuxSecurityLabel", info.sec_label, strlen (info.sec_label)+1);

Signed-off-by: INSUN PYO <insun.pyo@samsung.com>
Change-Id: Idff013ca4c56d93a6207bcb7365b36da86f7c1c2

gkdbus: Fix wrong return value of _g_kdbus_quit_loop()

_g_kdbus_quit_loop() which is used as a callback function in
g_main_context_invoke() always return 'TRUE'. This is not correct.
Returnning 'TRUE' means its functionality isn't completed. And it
results infinite loop time to time.

This patch fixes the function's return value correctly.

Change-Id: I08ea4c3a7c294774163c63983b48b458aa1d5c43
Signed-off-by: Jonghwa Lee <jonghwa3.lee@samsung.com>

Merge "kdbus: fix for GetConnectionSELinuxSecurityContext method call" into tizen

kdbus: fix build error for 64 bit architecture

Signed-off-by: INSUN PYO <insun.pyo@samsung.com>
Change-Id: I25c8313c5cf8e22702fe61703445a7640fad9298

kdbus: fix for GetConnectionSELinuxSecurityContext method call

To keep compatibility with dbus1, GetConnectionSELinuxSecurityContext
method call should always return SELinuxSecurityContextUnknown error
on systems without SELinux (even if other LSM systems, like for example
SMACK, are available on platform).

Change-Id: Iaff5afad798b06179c298e65955f90038882b54f

kdbus: enable KDBUS_MEMORY_POOL_SIZE environment to support adjustable memory pool

Signed-off-by: INSUN PYO <insun.pyo@samsung.com>
Change-Id: Ic593bc76914ef0aba3b448924d40a52e276df529

Merge "change KDBUS_INFINITE_TIMEOUT_NS to 0x3ff.. to avoid wraparound - will work for 144 years of uptime" into tizen

Merge "kdbus: always use well-known names for proxy" into tizen

change KDBUS_INFINITE_TIMEOUT_NS to 0x3ff.. to avoid wraparound - will work for 144 years of uptime

Change-Id: I597460ffb82d5551f221de0554c0f81746f5dee9

Detailed error information on KDBUS_CMD_SEND failures

Change-Id: Ied09920c30f014bcbfe2b7160fb9b13cc8297f09
Signed-off-by: Hyotaek Shim <hyotaek.shim@samsung.com>

kdbus: always use well-known names for proxy

If proxy->priv->name is a well-known name (what just means that we
created proxy for well-known name) then proxy->priv->name_owner points
to current name owner (unique id or NULL if name currently doesn't have
any owner). If we have following scenario:

1) server termination
2) g_dbus_proxy_call() to server
    2a) get_destination_for_call()
3) on_name_owner callback, which set proxy->priv->name_owner to NULL

then get_destination_for_call() function will return
proxy->priv->name_owner value (which is unique id of previous name
owner). To avoid above race condition it will be better if we 'always'
return well-known name (without checking name_owner value).

Change-Id: I31f7edb56d8434e26a15efe260c1b737486da93e

kdbus:remove not necessary(dbuplicated) code - kdbus timeout

Signed-off-by: INSUN PYO <insun.pyo@samsung.com>
Change-Id: Ia5a027ed30ccda95c3fa034fc793a9d2d8023526

Merge "global construction for dbusconnection only on _TV_PROFILE" into tizen

Revert (Build Error) : speed up incremental builds: %reconfigure instead of autogen

Signed-off-by: INSUN PYO <insun.pyo@samsung.com>
Change-Id: I6933e2aa5fca4d23a5c9c63ceb9d915081f9dc5d

kdbus: fixed(build error) memory leak in hello command - #2

Signed-off-by: INSUN PYO <insun.pyo@samsung.com>
Change-Id: I62a2b60968ba08409790b61bd670be9804334cbe

gkdbus reply timeout changes

- drop userspace timeouts on unix/kdbus (rely on kdbus instead)
- propagate user-provided timeouts to kdbus
- translate infinite timeouts (G_MAXINT) into 0x7fffffffffffffff ns
  (292 years)

Change-Id: Id769802925522cf82c26354dae9d8ee29fcb972d

Merge "modify signal subscription rule" into tizen

modify signal subscription rule

Change-Id: I8ab485f0a59e041ef735ff8cc74909cf9b7db91f
Signed-off-by: sanghyeok.oh <sanghyeok.oh@samsung.com>

Merge "kdbus: fixed memory leak in "hello" command." into tizen

speed up incremental builds: %reconfigure instead of autogen

Change-Id: I90aa6c14568fbefc981650c7c1beb56f26c66747

Merge "modify signal subscribtion rule" into tizen

kdbus: fixed memory leak in "hello" command.

Change-Id: I0e411bc9dbaf0b08bb622650596e1c491d85d3e5

modify signal subscribtion rule

Change-Id: I77ca7dc2222077ea0253aed7179582149b320b65
Signed-off-by: sanghyeok.oh <sanghyeok.oh@samsung.com>

Merge "kdbus: policy: add more detailed errors desciriptions" into tizen

Merge "Sync calls are now performed without lock held" into tizen

Sync calls are now performed without lock held

Sync calls were performed by function _g_kdbus_worker_send_message_sync()
called under connection lock. It was not necessary to hold the lock,
and it had side effects: holding another threads that wanted to call
something synchronically.

First argument of the function is connection->kdbus_worker. It is safe
to access this member, because it is read-only and outer context must keep
a reference to the connection, anyway. Worker itself is in turn kept
by reference from the connection.

The function uses only worker's read-only members, which are initialized
once at the creation of worker or connection. The only exception is
'matches' field, which is also modified in 'AddMatch'/'RemoveMatch'
method calls handler.

To sum up: I moved call to _g_kdbus_worker_send_message_sync() out of
connection lock and I have added a dedicated mutex for 'matches'.

Change-Id: I5895e591a9f32f085958b5b422d8b81110bd6c4d

modify signal subscribe rule to allow-none

modified to allow signal subscribe for NameOwnerChanged by 'allow-none' setting(sender, path, interface)

Change-Id: I94d57ad4ca5a7db0bc5f734275835d04fe99a540
Signed-off-by: sanghyeok.oh <sanghyeok.oh@samsung.com>

set KDBUS_DEFAULT_TIMEOUT_NS to 25 seconds

Change-Id: Ibff1b1c07d94a3b7679752d6c2fa1fe8fdd15d44

kdbus: policy: add more detailed errors desciriptions

Change-Id: I2bb44b798e2456809665205baec82d8436e935cd

global construction for dbusconnection only on _TV_PROFILE

Change-Id: Ie2430ed9210984ea012b37fa455229fc13895cdb
Signed-off-by: MyoungJune Park <mj2004.park@samsung.com>

Subject: [PATCH 1/2] kdbus: Fix the overflow for timeout calculation

The maximum value of unsigned long is 4294967295 which is too small
for storing nano second. It makes the overflow in calculation.

Use LLU instead of LU for timeout calculation.

======================================================================

Subject: [PATCH 2/2] kdbus: adjust the default value for timeout to 50 sec

This definition is used by the default value for timeout value.
But in the other places, 50 sec is used by default value for timeout,
instead of using it.

Let's align it with the others.

Signed-off-by: INSUN PYO <insun.pyo@samsung.com>
Change-Id: I9fb9839a311daf0c5839c87cdde793c2a4118869

svace:

[gio/gkdbus.c]
1. delete unreachable code
2. expand error check range

Change-Id: Ia36937b81d2e86111f9ce8a1f232ab0c842300e7
Signed-off-by: sanghyeok.oh <sanghyeok.oh@samsung.com>

Merge "kdbus: don't close the same memfd item twice" into tizen

Merge "kdbus: fix sending memfd items" into tizen

kdbus: don't close the same memfd item twice

Change-Id: I28e39b9dbbf58cb6fb16ba9721e5023208752777

Bug fix for signal hash(bloom filter) operations in gdbus-transport-kdbus

Signed-off-by: INSUN PYO <insun.pyo@samsung.com>
Change-Id: Ic051ee89a2e7bfb3c105be289082ebc5b6a94e88

kdbus: fix sending memfd items

Change-Id: Ibedc35ea0f3d443a8a3bde5151f145111a1eacf5

kdbus: catch up with latest libdbuspolicy API changes

Change-Id: I6bfb75785c7e817ad7a063f989671ab6d4279dc2

Merge "gdbus-codegen: avoid warnings when generated code is compiled with -Wunused-parameter" into tizen

Merge "Revert "Temporarily removing BuildRequires to libdbuspolicy to check cyclic build dep."" into tizen

Revert "Temporarily removing BuildRequires to libdbuspolicy to check cyclic build dep."

This reverts commit af3acf0be463fad4d470dbbab4b61b825edce0cf.

Change-Id: I5db890ca0e86f78d10736ec50ab82e0ab2f95c1d

gdbus-codegen: avoid warnings when generated code is compiled with -Wunused-parameter

Change-Id: Id62df4e7c7f7059a47ee7ee32aa054d91c6847a1

kdbus: don't check 'out' policy for method return

Change-Id: I7ac8abb8eebbc51d2dd20f632f0bc5d798fe4eae

kdbus: fix 'check in' policy

Change-Id: If3e39fc8fb9890c35f788a466346471785d578b8

Temporarily removing BuildRequires to libdbuspolicy to check cyclic build dep.

Change-Id: Ic4bd089219fc91ba3b514879be1232a4bf2dddad
Signed-off-by: Hyotaek Shim <hyotaek.shim@samsung.com>

Merge "Add BuildRequires to the libdbuspolicy package" into tizen

Add BuildRequires to the libdbuspolicy package

Change-Id: I2ff0b52b12b8ade8dcdf85427d23d2be2f26c027
Signed-off-by: Hyotaek Shim <hyotaek.shim@samsung.com>

kdbus: Fix memleak. Use proper kdbus device node path for session bus

Change-Id: Ic28fee3cf586d4f681ae35ec739fc95d023472c5

glib2:svace for gkdbus

Change-Id: Ieb227a765f3fa3fa8c3ad858beaaeb45aa01628a
Signed-off-by: sanghyeok.oh <sanghyeok.oh@samsung.com>

Minor fixes for build problems

Change-Id: Ia998daae96c4bd9db2b65f26d977928b36f3fd2e
Signed-off-by: Hyotaek Shim <hyotaek.shim@samsung.com>

Fix for several memory leak cases when using _g_dbus_enum_to_string()

Change-Id: I11a9e08c79ffaf052e21cf0501c13bcc56f6cb13
Signed-off-by: Hyotaek Shim <hyotaek.shim@samsung.com>

Temporarily removed BuildRequires for libdbuspolicy

Change-Id: I6b9b21dbd036896a8f5b4d635836c84cd639212e
Signed-off-by: Hyotaek Shim <hyotaek.shim@samsung.com>

Remove cynara-client from BuildRequires section

Change-Id: Ie9f9e1cf9900c24e1774e0ffde1892cbec52a39b

Merge "kdbus: support for libdbuspolicy" into tizen

fixed race condition between source and worker

Change-Id: I67cb04512aa971d60889aabf641fbbdd3383efdd

kdbus: support for libdbuspolicy

Change-Id: I634ed6d0eeb7a27df504624577de5c22d2806a69

Fixed license declaration at spec file

Change-Id: I0ef9ac5f36d9d59af84313bc469e4be16a0c82d3
Signed-off-by: DongHun Kwak <dh0128.kwak@samsung.com>

kdbus: bring back original gio semantic in d-bus name management

Since when we have 'dbus1 compatibility layer' we do not have to use
our new API, which offers only synchronous calls. Thanks of that we don't
have to modify gdbusnameowning.c and gdbusnamewatching.c files what causes
that GLib kdbus port is easier to maintain/rebase.

Change-Id: I76bf7b436e3dd82f6cd7e309b4a08b6b49363edd

kdbus: fix bugs and defects found by Klocwork and CPP Test. Add more logs.

- fix memleak in g_kdbus_setup_bloom() function;
- add logs for synthetic method calls and replies;
- fix body signature check in gkdbusfakedaemon;

Change-Id: I33d5f645ad97cddff8a562400ec28d2b907449fa

kdbus: properly attach KDBUS_ITEM_CANCEL_FD item

Change-Id: I0ca3220b08a3e988502cfc4e47687d0faeeecde8

fixed memory-leak - loophole

Change-Id: I205462ee6164bd336fa284e8076b327f15daadba

added newer kdbus return flags handling

Change-Id: I6ec98cc2ec82340d1f300a7d620e74a63b5b1640

kdbus: fix 'GetConnectionCredentials' synthetic reply builder

Change-Id: I24d19b567093dc9cffbe6b8093f3dc717353bd03

kdbus: Add support for GCanellable

Change-Id: Ie2caf15feb018690fbf9d5abffc85a4bfbd19a87
Signed-off-by: Lukasz Skalski <l.skalski@samsung.com>

gdbus: integrate kdbus into GDBus core

Patch includes also changes made by Ryan Lortie <desrt@desrt.ca>

https://bugzilla.gnome.org/show_bug.cgi?id=721861

Change-Id: I6bab6fee34787816efeb42d8e3e60c4c551a8a2b

gdbus: add kdbus core files

Patch includes also changes made by Ryan Lortie <desrt@desrt.ca>
and Adrian Szyndela <a.szyndela@samsung.com>

https://bugzilla.gnome.org/show_bug.cgi?id=721861

Change-Id: Ie90076a28c611320cd9b5ed6f2123d0ef85c6326

gdbus: import kdbus interface header

kdbus.h taken from https://github.com/systemd/kdbus, commit 9c04b6b989c9.

Hosting of our own copy of the kdbus header is temporary solution.
In the future, we should include it from the system kernel headers.

https://bugzilla.gnome.org/show_bug.cgi?id=721861

Change-Id: I2a42c91648b10c7c14f4df9ce2755f7c60524903

gvariant: substantial rework for kdbus purposes

This patch is 'squashed' version of Ryan Lortie's patches from
wip/kdbus-junk branch [1].

Main changes:
- add GVariantVectors utility struct,
- support serialising/deserialising to/from GVariantVectors,
- support for single precision floats,
- GVariantVectors serialisation tests,

[1] https://git.gnome.org/browse/glib/log/?h=wip/kdbus-junk

https://bugzilla.gnome.org/show_bug.cgi?id=721861

Change-Id: If70bd22337b5a9cddac545dce45474eba542d0de

glib-unix: add new internal header glib-linux.h

This defines some constants and syscall wrappers for features which may
not yet have appeared in the libc.

https://bugzilla.gnome.org/show_bug.cgi?id=721861

Change-Id: I7e83dadf8be9121f91fa0743bb8c1c612f923e13

glib-unix: add function to ensure an fd is sealed

Add a function that checks if a fd is sealed and, if it's not, seals it.

On Linux this is more or less an operation on memfd.  On other systems,
it currently always returns FALSE.

https://bugzilla.gnome.org/show_bug.cgi?id=721861

Change-Id: I021e40249c2cdd0f1137f6f9f3813f5d4e5083c5

gbytes: substantial rework for kdbus purposes

This patch is 'squashed' version of Ryan Lortie's patches from
wip/kdbus-junk branch [1].

Main changes:

- introduce a new type of GBytes: 'inline' -  this allows us to
  make a single allocation instead of two in the g_bytes_new() case,

- new g_bytes_take_zero_copy_fd() function - function takes a memfd,
  seals it, and creates a GBytes based on it,

- add g_bytes_get_zero_copy_fd() function - add a way to get the
  zero-copy fd back out of a GBytes that was created from one.

[1] https://git.gnome.org/browse/glib/log/?h=wip/kdbus-junk

https://bugzilla.gnome.org/show_bug.cgi?id=721861

Change-Id: I65c31e42c23346f3d7351f815ccbeda7461d3c01

packaging: Bump to 2.44.1

Change-Id: I1eb90fc7dc9d717e16d30985a4458f2a2dba223c
Signed-off-by: Lukasz Skalski <l.skalski@samsung.com>

packaging: Bump to 2.43.1

Change-Id: Id068a8288f85baef2ffada606d10bcb2dbf78cf4
Bug-Tizen: TC-2124
Signed-off-by: Philippe Coval <philippe.coval@open.eurogiciel.org>

packaging: Bump to 2.40.1

Change-Id: I23e399e680e58b26358c00e15cd4325c5eb08338
Signed-off-by: Philippe Coval <philippe.coval@open.eurogiciel.org>

packaging: Bump to 2.40.0

Change-Id: I6d64c12841cc6d0c536e27f1f3d69ec3a675d7d6
Bug-Tizen: PTREL-773
Bug-Tizen: TIVI-2898/part
Signed-off-by: Philippe Coval <philippe.coval@open.eurogiciel.org>

add .gbs.conf and update to 2.36.4

Change-Id: I0540f90b0b0ba28680cd5ae1d5186d07be078e61
Signed-off-by: Anas Nashif <anas.nashif@intel.com>

resetting manifest requested domain to floor

changes: Bug-Tizen: TZPC-3317 (... remove the fav apps icon ...)

Conflicts:
packaging/glib2.changes

Remove empty docs package

remove docs package

Signed-off-by: Anas Nashif <anas.nashif@intel.com>

Update to 2.36.1

Update to 2.36.0

Update to 2.35.9

Remove previous crash fix (TZPC-453)

Update to 2.35.8

Update to 2.35.8

Update to 2.35.7