API for comparing AtspiAccessible objects by unique_id

Change-Id: I925a9761013ac6656b1e09ac91cd79397b8b042f

Fix memory leak, handle hash table size

(1) Fix memory leak of atspi_accessible_get_reading_material.
(2) Remove defunct accessible from application hash table.
  In case of chromium-efl, it emits lots of "defunct" events without
  "RemoveAccessible", and this makes application hash table too big
  which could cause a Memory Alert on TV product.
(3) If AT-server does not emit "RemoveAccessible", then there is not
  a chance to remove accessible from the application hash table. And
  then the memory usage of AT-client grows bigger and bigger. So this
  patch is using newly defined value HASH_TABLE_SIZE_MAX.

Change-Id: If60d1fa1714d8bbf73b48cf16b1e1274f5a30cb9

fix memory leak in atspi_accessible_get_default_label_info

Change-Id: I8e6094e96381fbf276556c04a1f87e0cb656d1b3

Fix memory leak

* Release memory before pointer overwrite.
* Free memory before overwrite pointer.
* Memory leak with ref_accessible.
* Leak in atspi_state_type_get_type.
* Remove memory leak in
  atspi_event_listener_register_from_callback_full.
* Reduce memory leaks reported on program exit.
* This api is unused by efl but still let's fix it.

Change-Id: Ifde65dd44643f2e58442cf70809703f979436e09

Added printing toolkit name to at_spi2_tool

Change-Id: Ia8ab65788ff932a9371dfdf8a941693b2ac8b552

Added printing supported actions to at_spi2_tool

Change-Id: I15ba50fb0b06990d314a1f8ad554baf909c2312b

spec: update license from LGPL-2.0+ to LGPL-2.1+

Change-Id: If2866311d3282b9036c14459934d496a0ba9c7f9

Fix memory management issues

atspi/atspi-mutter.c

Dynamic memory referenced by 'name' was allocated at
atspi-accessible.c:307 by calling function 'atspi_accessible_get_name'
at atspi-mutter.c:398 and lost at atspi-mutter.c:401.

Pointer '&data.rd_session_path[0]' returned from function 'g_strdup' at
atspi-mutter.c:82 may be null, and it is dereferenced at
atspi-mutter.c:83.

atspi/atspi-event-listener.c

Return value of a function '_atspi_dbus_return_accessible_from_iter' is
dereferenced at atspi-event-listener.c:1094 without checking, but it is
usually checked for this function (5/6).

Change-Id: Ie311948f4d8f71025d0b1769a97e2904ee4af387

Fix upstream in .gbs.conf

Change-Id: Ia07c670b92bffc0e0d3ba0f7630ce8b88f5db8f1

atspi-misc: Prevent memory leak of main_loop pointer

atspi_main_loop is generated with g_main_loop_new
The loop ends and a memory leak occurs.
Call unref on atspi_main_loop to prevent leaks.

Change-Id: If8158f0113bf3eb31ff7b2558c835a92219a40d2

Merge tag 'upstream/2.34.0' into tizen

Change-Id: I480c8e572f5fe7d3d7d2d6f1f0aa2a76f009c9a8

2.34.0

Change-Id: I514111ff14f955ae7cc405bf8ff6298d7e982175

Apply systemd-based dbus activation

Change-Id: I3ccf53942b6c49fbe656e2e4387cc681809c2c98

Resolve the possible use of negative value (detected by coverity tool)

Change-Id: Ibb68f024ef14a0beb10d2bc63ff28fe03d55762f

Merge "Revert "tizen: Add additional unit for "unified" user session"" into tizen

fix for potential resource leak

fix for potential memory leak, where pointer memory allocated and
stored in a11y_bus_address property could be overwritten
without releasing.

Change-Id: I0835ec807c96192cc011374b81e2eb14bc910c58

Revert "tizen: Add additional unit for "unified" user session"

This reverts commit c64ea0f61a0842c7b71c3ccb9018fd9dce088164.

We've optimized systemd-user without unified system/session.
So, we revert the patch for unified system/session.

Change-Id: I4790604bf3b41c484aef3ffd029719f04a92fbce

Merge "at_spi2_tool is now in path'ed directory" into tizen

at_spi2_tool is now in path'ed directory

Change-Id: Iee3957bc7259622191e09b2f06e6d2262e67faf5

Fix meson build error

Change-Id: If9f9fcbd94c9f562d239d4b134f97bfb3a4ea705

Merge "tizen: Add additional unit for "unified" user session" into tizen

tizen: Add additional unit for "unified" user session

The unified user session is about moving user session units, managed by
systemd --user, to main systemd, where it's managed as part of newly
created user@.target.

user@.target will contain same units as previously available in user/,
with same UID and environment setup.  systemd instance is used for unit
to be able to specify UID (inherited from user@.target).

The rationale behind this work is following:

* VD requirement to remove user session support
* boot time optimization requirements, due to:

   + 'systemd --user' taking 1s its own startup that could be used for
     unit startup
   + ability to better rearrange units if these managed by one systemd
     instance

Unit installed by this commit will not be used till user login mechanism
will be changed in systemd package (via changing pam_systemd to start
user@.target, rather than user@.service).

Change-Id: I232c8be6cc6fce07ae694d5b0ac2f08d8b98c3a0

Fixes SVACE issue with thread-unsafe calls to strerror

Calls to strerror were replaced with calls to strerror_r, which
is thread-safe.

Change-Id: I6ee4908672118248b96590f375005ee424daea85

Fix for missing -fPIE build flag

Adds -fPIE build flag to meson, which is required, because not ASLR
binaries won't be allowed to run on target.

Change-Id: I1f725eb4c7f39a32c43dd104255b16b0628d7a38

Fix for invalid at-spi configuration

Fixes not working configuration (dbus-daemon binary not found), which
caused at-spi2-bus-launcher not to start.
Fixes crash due to missing schema for GSetting.

Change-Id: I5f1692144b4c7487f33739832780224707be001c

resolve build error in aarch64 and x86_64

Change-Id: I7cb193c626a19a818ed0c232d3cc788866679c82

Merge upstream branch 2.31.1 into branch origin/tizen

Change-Id: I02fb778191626e1445e40c4477fc17f79ce2fcd0

2.31.1

Change-Id: I5608051794e244fbcec00fbfafb9a6d71f485254

Redefine universal-switch vconf keys.

Change-Id: Ib01e580d4b263af6694cd473a3c313d6cdfc8bd6

launch universal-switch, if any of associated vconf flags is set

Change-Id: I9efc756419d76bc79b8dd9a531a775f3eacdfe13

[prevent][73691] Fix for resource leak

Change-Id: I7030fa9b835457b72ca2b826df4e2e9942d69d34

[prevent][76366] Fix for resource leak

Change-Id: Ieaab30db6d430f898ab3959528bacd4056bfefd8

Merge branch 'upstream/2.26.1' into tizen

Change-Id: Iab32f00420decb25c5393dbfdf31bc6a1c7495bf

Add member of default label info that check for default label enabed

Change-Id: I9f20e4aed82de71c5f64210817f8b11f6505f63d

[prevent][38358] Fix for resource leak

Change-Id: I26e7404fa047899f55d4150e2ed427aa097a681f

[prevent][33973] Fix for unchecked return value

Change-Id: I528adb092e1f8361dfbcb95ecc2d307587e26aac

[prevent][38682] Fix for resource leak

Change-Id: Iea61c399f59d2ddb2b55768f325c3005e01cfcf3

[prevent][40806] Fix for dereference after null check

Change-Id: Ic7239f40dcd5270a6fa4edf95daf30459ebfcdd4

[prevent][24443] Fix for resource leakage

Change-Id: Ie4377a46c1d96e2a88da9851e72e6aa7b5198d09

[prevent][35992] Fix for resource leakage

Change-Id: Iee9baa0546c1aee54cd9fbea4e37b63360767759

[prevent][23957] Remove logically dead code

Change-Id: I63b4a859375ba2813b6805db8d00521a41fe699e

[prevent][36018] Fix for unchecked return value

Change-Id: I79eba8a9907b44a0ea349eaab0ab2704284a200f

[prevent][35766] Fix for unchecked return value

Change-Id: I5138fda2b7128e8f8bf06621fc6ec4086829d5c0

[prevent][38655] Fix for unchecked return value

Change-Id: I117d7d12d0c2aa9f0ab32c239341ed2d91a5c14a

[prevent][25425] at_spi2_tool: make sure app_name is set in _atspi_tree_traverse

"prevent" bug 25425 is a false positive, but it's a good practice
to check function arguments before using them

Change-Id: I884803613b670d1126120a689bb21ac78e87b8ab

[prevent][43479] Fix for missing va_end in _atspi_dbus_call_partial

Change-Id: Ice71e5aaa543ca7908464538f15f3fbd921c464f

[prevent][42662] Fix for unchecked return value

Change-Id: Iee850e8fb485800d5c045fe4c7cd81728defd1d6

[prevent][42562] Fix for resource leakage

Change-Id: Iabf4421164e91f6bcafe3b3be43932e0cf4de098

[prevent][32379] Fix for resource leakage

Change-Id: I901af1e8739151b1d13b38c7d596678e1a286704

[prevent][42870] Fix for null pointer dereference

Change-Id: I74e51363d0e6ec34e9b771f605d283f1311af803

[prevent][36724] Fix for resource leakage

Change-Id: Ib0407cf3cf1de81e7c78c9d969daa067e0af428a

[prevent][109656] Fix for resource leakage

Change-Id: Ic2abd2a7c7eafd530d147711d8881fb87788e140

Fixes glib runtime warnings on freeing empty arrays

Change-Id: Iba679a3cfbcdcdfd252c291d5f8bcf433c4d7198

2.26.1

meson: Actually use the soversion

meson: x11 fixes

Meson wasn't defining HAVE_X11, so x11 support wasn't being compiled even
if X was determined to be present.
Also, add an enable-x11 option (default is to auto-detect).

Merge "test : enhance accessible state information" into tizen

test : enhance accessible state information

Change-Id: I112a34525f66e204aa5dbab51ad0faadbf25d3da

test: enhance accessible tree information

Enhance node information using eo address expression

Change-Id: I02e057013812073e7b7484cf3d87516f6a9d9053

[l10n] Updated Catalan (Valencian) translation

Add atspi_accessible_get_default_label_info

It is not possible to support default label feature using current at-spi2-core
APIs without much of IPC. The following would be difficult case to cover.

(top of accessible tree) - (bottom side)
PageTab1 - Panel1 - PageTab2 - Panel2  -  PageTab3 - Panel3(currently showing)

Application could make as below:
PageTab1 - Panel1 - PageTab3 - Panel3  -  PageTab2 - Panel2(currently showing)

or following tree would normally be possilbe:
PageTab1 - Panel1 - PageTab2 - Panel2(currently showing)

There are much of complicated case over there.
So we are handling the default label object stack on toolkit(Elementary) side.

The atspi_accessible_get_default_label_info returns default label information
to be used screen-reader side. This is not stable. And this depends on toolkit
side UI definition. The candidate of default label object could be changed by
UI definition.

AtspiAccessibleDefaultLabelInfo *dli;
dli = atspi_accessible_get_default_label_info(root, &error);

You have to free alocated resource as below, if it is not necessary any more.
g_object_unref(dli->obj);
free(dli);

Change-Id: I444906e6d020ea6fe2536a73473e0e796a015d05

Remove dependency on libxkbcommon

Currently, no functions in libxkbcommon are used, so the dependency is
spurious. Instead, check for XkbGetMap in -lX11.

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

Fix crash, when dbus call in GetNeighbor fails

Failed dbus call would cause a crash by trying to get an iterator
to a reply message, which is null. This patch cleans up
_atspi_dbus_call_partial function to always return NULL on failure
and fixes NULL reply handling in atspi_accessible_get_neighbor and
atspi_accessible_get_navigable_at_point functions.

Change-Id: Ie158a79f8f1452c9a5685137c9b42104563dc717

Merge "check reply message of "GetNeighbor"" into tizen

check reply message of "GetNeighbor"

The dbus_message_iter_init makes a crash if a message(reply) is NULL.
The message(reply) is NULL with following error message.

error message: Did not receive a reply. Possible causes include: the remote
application did not send a reply, the message bus security policy blocked the
reply, the reply timeout expired, or the network connection was broken.

Change-Id: I64c9f468f26299af86ee52dff3714b03112d49e3

Updated Dutch translation Master 3.26

2.26.0

bugfix: at-spi-bus-launcher terminates when unviersal-switch vconf key not defined

Change-Id: I4a891f300aeb4261938a9e5dbdf4cdf5749c7d05

2.25.92

Optionally read the dbus address from env var AT_SPI_BUS_ADDRESS

This will be used for flatpak to set a custom bus (which is really
the bus proxy). It can be used for testing purposes too.

For more details, see https://github.com/flatpak/flatpak/issues/79

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

at-spi-bus-launcher: launch universal-switch

Change-Id: I16bff1e30b2c72c812be767f1b21f88a65ee1787

Update MAINTAINERS

Email address for me was old, and Li and Mark haven't been involved
for several years now.

configure.ac: make xkb optional, as intended

[] is not sufficient to avoid PKG_CHECK_MODULES' default action of bailing if
xkbcommon-x11 is not found.

Updated Danish translation

2.25.91

autoconf: include missing meson build files

meson: build translations

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

Update Malayalam translation

at_spi2_tool: Apply ASLR to executable files.

Change-Id: I86b9dd51723548d1b888eed7847668ca9e318110
Signed-off-by: Bowon Ryu <bowon.ryu@samsung.com>

meson: fix org.a11y.atspi.Registry.service

The service.in file references LIBEXECDIR, but meson was setting libexecdir.
Use libexecdir for consistency.

move ref_accessible function to public scope

this function is needed outside at-spi library (other projects, e.g. universal-switch),
to get AtspiAccessible object after it has been received as application name and path name

Change-Id: I51e72af3f191d4a14174cd0b4819522154e3e1ba

More meson fixes

2.25.90

meson: add support for gobject-introspection

Not 100% sure that this is done. The shared-library line seems wrong.

Add Nepali translation

Decrements the reference count of session_bus

dbus_bus_get increments reference count of a DBusConnection.
It is callers responsibility to all dbus_connection_unref after use.

bus: follow D-bus policy

Tizen D-bus policy does not support Smack rule.
The following patch set of D-bus probably affects dbus-daemon which is used for
accessibility.

https://review.tizen.org/gerrit/#/c/133270/

It was not possilbe to launch dbus-daemon with following error message.

org.a11y.Bus[2457]: Failed to start message bus:
Attribute "smack" is invalid on <policy> element in this context
org.a11y.Bus[2457]: Failed to launch bus: Bus exited with code 1

Change-Id: I67435b1a27f91193348d1e699805b6d09be8fddc

add attributes table to at_spi2_tool

atributes table will be printed, when atributes string is too long

add relations_found flag which tells if relation table legend should be printed

Change-Id: I2a0b71f5e8b439881f5b83e9b7f1a17556f9e713

Add relation-dump option to at_spi2_tool

Flags -d and -c now inform when object has relations to list.

Change-Id: I0c9a03c8d18ddf965c72ceac2bb804742c4540bf

Fix a couple of introspection issues

This mainly restores the gir entry for AtspiApplication.

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

atspi_action_do_action_name: change parameter type

The name is not changed in atspi_action_do_action_name.
So the type of name should be const gchar*

Change-Id: I6a9329ea34ef566dd444cae5112dead99ef4a9ce

Merge "Add MoveOuted signal" into tizen

Merge "Support many at-spi clients" into tizen

Add NEIGHBOR_SEARCH_MODE_RECURSE_TO_OUTSIDE mode

The "GetNeighbor" method returns an accessible object which has same bus with
root object. But the returned recurse value is 1.

The following is the case.
If embedded object has highlight, and there is no more previous object, then
it returns PROXY object which has embedding side bus information.

In this case, the "GetNeighbor" should find previous sibling object in embedding
process. The "GetNeighbor" is using NEIGHBOR_SEARCH_MODE_RECURSE_TO_OUTSIDE for
this case.

Change-Id: Icae2709b2746bd970643034c2d66b1c196c65ca9

Use deputy object sent by "GetNavigableAtPoint"

The deputy means embedding side object works for embedded object.
If there is no object at point on embedded side, then the deputy object should
have highlight.

So far the deputy object is elm_access, and the elm_access takes highlight.
So there is no chance to have highlight on embedded side.

This patch set reuires:
elementary: b82beff73c41f52542d71fd6f4d9ad4ddcb4bacc

Change-Id: I2136242b73883757bd50331b10f3e1f8a6bff986

Add atspi_accessible_get_reading_material

To get reading material to be used screen-reader side. This is not stable.
You have to handle all alocated memory as below on screen-reader side.

AtspiAccessibleReadingMaterial *rm
g_object_unref(rm->parent);
g_object_unref(rm->described_by_accessible);
g_hash_table_unref(rm->attributes);
free(rm->name);
free(rm->labeled_by_name);
free(rm->text_interface_name);
free(rm->localized_role_name);
free(rm->description);
free(rm);

Related patch set:
https://review.tizen.org/gerrit/#/c/131358/ (this one, at-spi2-core)
https://review.tizen.org/gerrit/#/c/131359/ (elementary)
https://review.tizen.org/gerrit/#/c/131511/ (at-spi2-atk)
https://review.tizen.org/gerrit/#/c/131714/ (screen-reader)

Change-Id: I968b058098162efe1cbec309411492ca29f696c8

Reduce action related IPC

If there are more than 10 actions on an object, then more than 10 IPC occurs for
doing more than 10th action.

Change-Id: Id3767be2978357d147e7f2ecf96aee8e48641c16

Add navigation helper functions for screen-reader and friends (part 2)

Change-Id: I67eb6da5262a9de721dc2cb569255079dc221ec9

Add navigation helper functions for screen-reader and friends

This patch adds GetNavigableAtPoint accessibility interface - the
function finds accessibility object at given coordinates.
This massively (10-15 times) reduce amount of IPC calls (and time spent)
in typical screen-reader scenario.

Requires:
- https://review.tizen.org/gerrit/c/117306/ (elementary)

Change-Id: I0515e220f451fb196c0e346ea8e2dbd6d0d7d02f

Extends support for uniquely identyfing objects.

- adds atspi_accessible_get_unique_id function, which returns unique
identifier across all bridges,
- and atspi_accessible_get_bus_name function, which returns identifier
of the bridge for given object.

Change-Id: Idccd85e3269e5a36c8d96bc8588f4eac175c52f6