| Commit message (Collapse) | Author | Age | Files | Lines |
|\
| |
| |
| |
| |
| |
| |
| |
| | |
* upstream/pull/147
IlmControl: fix no warning in a function
Reviewed-by: Eugen Friedrich <efriedrich@de.adit-jv.com>
Reviewed-by: Tran Ba Khang <khang.tranba@vn.bosch.com>
Tested-by: Doan Ngoc Au <au.doanngoc@vn.bosch.com>
|
| |
| |
| |
| |
| |
| |
| |
| | |
In function input_listener_input_acceptance, variable "surface_found"
should be initialized to 0. Only then "if(!surface_found)" will take
effect.
Signed-off-by: Au Doan Ngoc <au.doanngoc@vn.bosch.com>
|
|\ \
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
* upstream/pull/125
ivi-input-controller: ivi-controller: Copy over weston internal functions
Reviewed-by: Eugen Friedrich <efriedrich@de.adit-jv.com>
Reviewed-by: Harsha M M <harsha.manjulamallikarjun@in.bosch.com>
Tested-by: Doan Ngoc Au <au.doanngoc@vn.bosch.com>
|
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
Both ivi-controller and ivi-input-controller access functions which are
internal to weston, which means the ABI of these functions may change or
they may even be completely removed. So far, this wayland-ivi-extension
was able to link against libweston and use those functions by sheer luck,
but that may eventually run out. Copy over the internal functions as of
weston commit 2c1ed289 ("build: bump to version 9.0.92 for the beta release")
Signed-off-by: Marek Vasut <marex@denx.de>
|
|\ \ \
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
* upstream/pull/141
CMakeLists: update libweston to version 8
Remove unused function decodeEscapesequences
Remove unused function importSceneFromTXTHelper
Add missing include of endian.h
Fix crash in controller_screenshot_notify
Reviewed-by: Eugen Friedrich <efriedrich@de.adit-jv.com>
Reviewed-by: Harsha M M <harsha.manjulamallikarjun@in.bosch.com>
Tested-by: Doan Ngoc Au <au.doanngoc@vn.bosch.com>
|
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
Update the libweston-8 on ivi-id-agent cmake file.
Update the libweston-8-protocols on simple-weston-client cmake file
Signed-off-by: Tran Ba Khang(MS/EMC31-XC) <Khang.TranBa@vn.bosch.com>
|
| | | | |
|
| | | | |
|
| | | |
| | | |
| | | |
| | | | |
htole32, htole16 needs endian.h to be included.
|
|/ / /
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
With the following change in weston-8 the wl_signal_emit's data parameter has been changed from output to output_damge:
https://github.com/wayland-project/weston/commit/0bb94476532ebbe95e255a9a736ee0c48866db53 has changed
wl_signal_emit(&output->frame_signal, output);
=>
wl_signal_emit(&output->frame_signal, output_damage);
The function controller_screenshot_notify in ivi-controller.c interprets the data parameter as output, which causes winston to crash.
With this chage, the output is now stored in the screenshot_frame_listener struct and controller_screenshot_notify uses the stored value, instead of the signal's data parameter.
|
|\ \ \
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
* upstream/pull/146
ivi-id-agent: Revert the weston header link
Reviewed-by: Eugen Friedrich <efriedrich@de.adit-jv.com>
Tested-by: Doan Ngoc Au <au.doanngoc@vn.bosch.com>
|
|/ / /
| | |
| | |
| | |
| | |
| | |
| | | |
The WESTON_INCLUDE_DIRS contains a link to ${install_dir}/include/weston.
So, header files should not include the weston directory in the path.
Signed-off-by: Tran Ba Khang(MS/EMC31-XC) <Khang.TranBa@vn.bosch.com>
|
|\ \ \
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
* upstream/pull/110
simple-weston-client: update dependencies to build on weston 7
ivi-id-agent: update dependencies to build on weston 7
ivi-id-agent: update to weston 7 header files
ivi-input-controller: update to weston 7 header files
Reviewed-by: Eugen Friedrich <efriedrich@de.adit-jv.com>
Reviewed-by: Harsha M M <harsha.manjulamallikarjun@in.bosch.com>
Tested-by: Doan Ngoc Au <au.doanngoc@vn.bosch.com>
solved merged onflicts:
ivi-id-agent-modules/ivi-id-agent/src/ivi-id-agent.c
|
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
upgrade libweston-protocols version.
Signed-off-by: Rajendraprasad K J <KarammelJayakumar.Rajendraprasad@in.bosch.com>
|
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
upgrade libweston-desktop version.
Signed-off-by: Rajendraprasad K J <KarammelJayakumar.Rajendraprasad@in.bosch.com>
|
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
update the header file path.
Signed-off-by: Rajendraprasad K J <KarammelJayakumar.Rajendraprasad@in.bosch.com>
|
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
update the header file path.
Signed-off-by: Rajendraprasad K J <KarammelJayakumar.Rajendraprasad@in.bosch.com>
|
|\ \ \ \
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
* upstream/pull/140
README: update for wayland backend
Reviewed-by: Eugen Friedrich <efriedrich@de.adit-jv.com>
|
| | |_|/
| |/| |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
On Ubuntu 22.04, the display controller are using the wayland protocols
by default. Run a new weston process need to pass the wayland display
socket listening. Clients want to run with new weston process need to
set WAYLAND_DISPLAY environment variable.
Signed-off-by: Tran Ba Khang(MS/EMC31-XC) <Khang.TranBa@vn.bosch.com>
|
|\ \ \ \
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
* upstream/pull/139
multi-touch-viewer: add the type of shell selection input
Reviewed-by: Eugen Friedrich <efriedrich@de.adit-jv.com>
Reviewed-by: Harsha M M <harsha.manjulamallikarjun@in.bosch.com>
Tested-by: Doan Ngoc Au <au.doanngoc@vn.bosch.com>
|
| |/ / /
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
The wl_shell and ivi_shell can’t create a surface base on a wl_surface at
the same time, so a selection of shell before running the example is required.
The ivi_shell is the shell default,
passing the wl_shell follow by –s option to select wl_shell
Signed-off-by: Tran Ba Khang(MS/EMC31-XC) <Khang.TranBa@vn.bosch.com>
|
|\ \ \ \
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
* upstream/pull/138
ivi-id-agent: Fix invalid read reported by valgrind while stopping Weston
Reviewed-by: Eugen Friedrich <efriedrich@de.adit-jv.com>
Reviewed-by: Harsha M M <harsha.manjulamallikarjun@in.bosch.com>
Tested-by: Doan Ngoc Au <au.doanngoc@vn.bosch.com>
|
| |/ / /
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
Valgrind reports the following issue while stopping Weston.
==2459== Invalid read of size 8
==2459== at 0x7169040: deinit (ivi-id-agent.c:369)
==2459== by 0x488E98F: wl_signal_emit (wayland-server-core.h:394)
==2459== by 0x488E98F: weston_compositor_tear_down (compositor.c:7686)
==2459== by 0x40681F: main (main.c:3397)
The elements of application list are not properly destroyed as it's link to the
list exists even after deletion.
Remove the element link using wl_list_remove() and use wl_list_for_each_safe()
to safely iterate through the list.
Signed-off-by: Rajendraprasad K J <KarammelJayakumar.Rajendraprasad@in.bosch.com>
|
|\ \ \ \
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
* upstream/pull/136
EGLWLInputEventExample: fix crash when exit
Reviewed-by: Eugen Friedrich <efriedrich@de.adit-jv.com>
Reviewed-by: Harsha M M <harsha.manjulamallikarjun@in.bosch.com>
Tested-by: Doan Ngoc Au <au.doanngoc@vn.bosch.com>
|
| |/ / /
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
The m_wlCursor is allocated on wl_cursor_theme_load, and
free on wl_cursor_theme_destroy. The app don't need to manage
this resource.
Signed-off-by: Tran Ba Khang(MS/EMC31-XC) <Khang.TranBa@vn.bosch.com>
|
|\ \ \ \
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
* upstream/pull/135
ivi-id-agent-modules: missing linker directory include for libweston-desktop
CMake: Fix weston header link
CMake: correct the pkg_config header variable
Reviewed-by: Eugen Friedrich <efriedrich@de.adit-jv.com>
Reviewed-by: Harsha M M <harsha.manjulamallikarjun@in.bosch.com>
Tested-by: Doan Ngoc Au <au.doanngoc@vn.bosch.com>
|
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
If user only build weston, the wayland and pixman are
installed via apt, they will get the missing libweston-desktop header.
Signed-off-by: Tran Ba Khang(MS/EMC31-XC) <Khang.TranBa@vn.bosch.com>
|
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
The WESTON_INCLUDE_DIRS contains a link to ${install_dir}/include/weston.
So, header files should not include the weston directory in the path.
Signed-off-by: Tran Ba Khang(MS/EMC31-XC) <Khang.TranBa@vn.bosch.com>
|
| |/ / /
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
pkg_check_modules will create the include directory variable
of the module. The format is: ${MODULE}_INCLUDE_DIRS.
Signed-off-by: Tran Ba Khang(MS/EMC31-XC) <Khang.TranBa@vn.bosch.com>
|
|\ \ \ \
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
* upstream/pull/133
protocol: Link ivi-application library with wayland-client library
Reviewed-by: Eugen Friedrich <efriedrich@de.adit-jv.com>
Reviewed-by: Harsha M M <harsha.manjulamallikarjun@in.bosch.com>
Tested-by: Doan Ngoc Au <au.doanngoc@vn.bosch.com>
|
| |/ / /
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
ivi-application library uses wayland-client library symbol:
aarch64-linux-gnu-objdump -TC libivi-application.so | grep wl_
0000000000000000 D *UND* 0000000000000000 wl_surface_interface
but couldn't observe it in the library's 'Dynamic Section':
aarch64-linux-gnu-objdump -p libivi-application.so | grep NEEDED
NEEDED libc.so.6
Link libivi-application.so with libwayland-client.so to prevent undefined symbol
errors.
Signed-off-by: Rajendraprasad K J <KarammelJayakumar.Rajendraprasad@in.bosch.com>
|
|\ \ \ \
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
* upstream/pull/128
ilm: remove declaration of a global variable in a header file
Reviewed-by: Eugen Friedrich <efriedrich@de.adit-jv.com>
Reviewed-by: Harsha M M <harsha.manjulamallikarjun@in.bosch.com>
Tested-by: Doan Ngoc Au <au.doanngoc@vn.bosch.com>
|
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
fixes following error with the gcc 11.2.0:
usr/bin/ld: CMakeFiles/ilmCommon.dir/src/ilm_common_wayland_platform.c.o:(.bss+0x0): multiple definition of `gIlmCommonPlatformFunc'; CMakeFiles/ilmCommon.dir/src/ilm_common.c.o:(.bss+0x0): first defined here
collect2: error: ld returned 1 exit status
make[2]: *** [ivi-layermanagement-api/ilmCommon/CMakeFiles/ilmCommon.dir/build.make:119: ivi-layermanagement-api/ilmCommon/libilmCommon.so.2.2.0] Error 1
make[1]: *** [CMakeFiles/Makefile2:511: ivi-layermanagement-api/ilmCommon/CMakeFiles/ilmCommon.dir/all] Error 2
ilm_client and ilm_common both has the same issue.
fix: move the declaration to the c file and define the variable as "external"
in the header file
Signed-off-by: Jeka Friedrich <friedrix@gmail.com>
|
|\ \ \ \ \
| |_|/ / /
|/| | | |
| | | | |
| | | | | |
khangtb1/fix_crash_and_hang_on_simple_weston_client_example
simple-weston-client: fix crash and hang when exiting
|
|/ / / /
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
wlcontext->debug_iface didn't bind when weston has been running
with non debug option. So, we need to check it before call
weston_debug_v1_destroy.
Need to remove the while loop after send SIGINT, the stream->ojb
always available in this case, then it makes a hang here.
Signed-off-by: Tran Ba Khang(MS/EMC31-XC) <Khang.TranBa@vn.bosch.com>
|
|\ \ \ \
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
* upstream/pull/124
EGLWLInputEventExample: Repair array out of bounds access
simple-weston-client: Repair array out of bounds access
Reviewed-by: Eugen Friedrich <efriedrich@de.adit-jv.com>
Tested-by: Eugen Friedrich <efriedrich@de.adit-jv.com>
|
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
The problem is in the `fprintf(..., left_ptrs[j])`, where if this code is
ever triggered, the variable `j=4` always, while the `left_ptrs` array only
has four entries instead of five, so the code would access one entry past
the array. In case this code is triggered, it does cause segmentation fault.
Signed-off-by: Marek Vasut <marex@denx.de>
|
| |/ / /
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
Fix array out of bounds access detected by gcc format fortification check:
```
In file included from /test/recipe-sysroot/usr/include/stdio.h:867,
from /test/git/ivi-layermanagement-examples/simple-weston-client/src/simple-weston-client.c:23:
In function 'fprintf',
inlined from 'create_cursors' at /test/git/ivi-layermanagement-examples/simple-weston-client/src/simple-weston-client.c:120:9,
inlined from 'seat_handle_capabilities' at /test/git/ivi-layermanagement-examples/simple-weston-client/src/simple-weston-client.c:247:9:
/test/recipe-sysroot/usr/include/bits/stdio2.h:100:10: error: '%s' directive argument is null [-Werror=format-overflow=]
100 | return __fprintf_chk (__stream, __USE_FORTIFY_LEVEL - 1, __fmt,
| ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
101 | __va_arg_pack ());
| ~~~~~~~~~~~~~~~~~
```
The problem is in the `fprintf(..., left_ptrs[j])`, where if this code is
ever triggered, the variable `j=4` always, while the `left_ptrs` array only
has four entries instead of five, so the code would access one entry past
the array. In case this code is triggered, it does cause segmentation fault.
Signed-off-by: Marek Vasut <marex@denx.de>
|
|\ \ \ \
| |/ / /
|/| | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
* upstream/pull/107
simple-weston-client: remove unused variable
simple-weston-client: use correct dlt header
simple-weston-client: make dlt optional
Reviewed-by: Eugen Friedrich <efriedrich@de.adit-jv.com>
|
| | | |
| | | |
| | | |
| | | | |
Signed-off-by: Jeka <friedrix@gmail.com>
|
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
dlt.h can be used instead of dlt_common.h and dlt_user.h
Signed-off-by: Jeka <friedrix@gmail.com>
|
| |/ /
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
if weston-debug protocol is available we should use dlt
if it is available so only optional otherwise we will just
output to stderr
Signed-off-by: Eugen Friedrich <efriedrich@de.adit-jv.com>
|
|\ \ \
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
* upstream/pull/122
ivi-input-controller: Prevent use-after-free when tearing down seat
Reviewed-by: Eugen Friedrich <efriedrich@de.adit-jv.com>
|
| | |/
| |/|
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
The seat destroy callback is triggered from weston_seat_release(),
that means the only way the code can enter destroy_seat() is with
all input devices associted with seat already released and all the
pointers already invalid. Avoid touching any of these input device
pointers in the destroy_seat() callback, otherwise this triggers
use-after-free condition.
ASan detects the use-after-free condition as follows:
```
==89420==ERROR: AddressSanitizer: heap-use-after-free on address 0x61100028e6a0 at pc 0x7f50deacb6fd bp 0x7ffed8504fe0 sp 0x7ffed8504fd8
WRITE of size 8 at 0x61100028e6a0 thread T0
#0 0x7f50deacb6fc in weston_keyboard_end_grab ../libweston/input.c:1604
#1 0x7f50bc6aac97 in destroy_seat (/tmp/wt/lib/x86_64-linux-gnu/weston/ivi-input-controller.so+0x7c97)
#2 0x7f50bc6ab02f in handle_seat_destroy (/tmp/wt/lib/x86_64-linux-gnu/weston/ivi-input-controller.so+0x802f)
#3 0x7f50deac477d in wl_signal_emit /usr/include/wayland-server-core.h:478
#4 0x7f50dead5a93 in weston_seat_release ../libweston/input.c:3490
#5 0x7f50bc6973d0 in ss_seat_destroy ../compositor/screen-share.c:395
#6 0x7f50bc69922c in registry_handle_global_remove ../compositor/screen-share.c:751
#7 0x7f50de8b89d9 (/lib/x86_64-linux-gnu/libffi.so.8+0x79d9)
#8 0x7f50de8b7b20 (/lib/x86_64-linux-gnu/libffi.so.8+0x6b20)
#9 0x7f50dea582d1 (/lib/x86_64-linux-gnu/libwayland-client.so.0+0x92d1)
#10 0x7f50dea54979 (/lib/x86_64-linux-gnu/libwayland-client.so.0+0x5979)
#11 0x7f50dea5603b in wl_display_dispatch_queue_pending (/lib/x86_64-linux-gnu/libwayland-client.so.0+0x703b)
#12 0x7f50bc69931a in shared_output_handle_event ../compositor/screen-share.c:771
#13 0x7f50dea434b1 in wl_event_loop_dispatch (/lib/x86_64-linux-gnu/libwayland-server.so.0+0xb4b1)
#14 0x7f50dea41134 in wl_display_run (/lib/x86_64-linux-gnu/libwayland-server.so.0+0x9134)
#15 0x7f50deed8489 in wet_main ../compositor/main.c:3564
#16 0x55961496217d in main ../compositor/executable.c:33
#17 0x7f50dece97ec in __libc_start_main ../csu/libc-start.c:332
#18 0x559614962099 in _start (/tmp/wt/bin/weston+0x1099)
```
Signed-off-by: Marek Vasut <marex@denx.de>
|
|\ \ \
| |/ /
|/| |
| | |
| | |
| | |
| | | |
* upstream/pull/121
Fix NULL pointer exception in case of no input device
Reviewed-by: Eugen Friedrich <efriedrich@de.adit-jv.com>
|
|/ /
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
When weston with wayland-ivi-extension run without input device,
that crash by NULL access at setup_input_focus function.
In this case, st_focus->seat_ctx is NULL pointer, but it use
without NULL check in this function.
This patch add NULL check for st_focus->seat_ctx.
Signed-off-by: Naoto Yamaguchi <naoto.yamaguchi@aisin.co.jp>
|
|\ \
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
* upstream/pull/120
test : Fix compiler warnings
EGLWLMockNavigation : Fix compiler warnings
simple-weston-client : Fix compiler warnings
ivi-controller : Fix compiler warnings
Reviewed-by: Eugen Friedrich <efriedrich@de.adit-jv.com>
|
| | |
| | |
| | |
| | |
| | |
| | | |
Solve indentation warnings for if-else case and for loop.
Signed-off-by: Shivakumar Halagatti <Shivakumar.Halagatti@in.bosch.com>
|
| | |
| | |
| | |
| | |
| | |
| | | |
Solve "ignoring return value" warning of fread function.
Signed-off-by: Shivakumar Halagatti <Shivakumar.Halagatti@in.bosch.com>
|
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
Solve "ignoring return value" of read and pipe functions and remove
unused variables temp and offset.
Signed-off-by: Shivakumar Halagatti <Shivakumar.Halagatti@in.bosch.com>
|