Add gitreview file for Royal Ricefish branch

Update the .gitreview file.

Bug-AGL: SPEC-5194
Signed-off-by: Jan-Simon Moeller <jsmoeller@linuxfoundation.org>
Change-Id: Ie13fb93ef169eeac35f0d75114c84bd7a87f2368

Add gitlab issue/merge request templates

Add template files for gitlab

Bug-AGL: SPEC-4474
Signed-off-by: Jan-Simon Moeller <jsmoeller@linuxfoundation.org>
Change-Id: I21a6bd78084716648cc184ec3485ce53dae119b6

gcc 11.x fixes

Changes for compiling with gcc 11.x:
- g++ now seems to instantiate duplicate entries for the set member
  function in the contextclass template class in binding-wrap.hpp.
  The use of a closure as a default argument value seems to be the
  culprit, as it seems there are longstanding issues with respect to
  using closures like that and resulting symbol names (i.e. the use
  of a closure isn't necessarily recognized as generating unique
  instantiations).  In theory, C++17 should explicitly allow this
  when the closure has no captures, but bumping up to -std=gnu++17
  did not fix the issue.  To avoid it, replace the closure usage
  with a private static member function.
- In afb-hook.c, tweaked the ignoring of the writev return code to
  make the stricter checking in gcc 11 happy.
- In decode_base64 in wrap-json.c, initialize u16 as gcc now seems
  to miss that it will be initialized on the first loop iteration.

Bug-AGL: SPEC-3819

Signed-off-by: Scott Murray <scott.murray@konsulko.com>
Change-Id: I8876db196b46cc35ecd2798e20d5ec3425df865e

CMake: Link with -latomic

Some architecutres (such as RISC-V) require that atomics be specifically
linked to avoid build failures. Ensure that this is the case.

Signed-off-by: Alistair Francis <alistair.francis@wdc.com>
Change-Id: Iad543232a6547b197f04100e996e6200c9f803ce

fix(migration-to-binding-v3.sed): add a legacy call replacing rule

afb_event_drop is not supported in appfw v3, use afb_event_unref to
replace it.

Bug-AGL: SPEC-2745

Change-Id: I6f2a1148c420fb9964eb94e5ae012be563d722e6
Signed-off-by: Li, Xiaoming <lixm.fnst@cn.fujitsu.com>

Fix typo

Bug-AGL: SPEC-3476

Signed-off-by: Zhang Nan <zhangnan.fnst@cn.fujitsu.com>
Change-Id: Icafb76c1cee6fb85e4f5fb95badb521fb96d45d5

Fix includes of C++ header

The include of <string> was missing in the C++ header.
This produced errors on some builds.

Bug-AGL: SPEC-3358

Signed-off-by: José Bollo <jose.bollo@iot.bzh>
Change-Id: I928e8ab267d27710177a7a314ea8b8cdfa83e2f3

afb-stub-ws: fix bug in reconnection

When the client reconnected, it used the wrong type
and reconnected as server leading to unexpected
continuations.

Bug-AGL: SPEC-3272

Signed-off-by: José Bollo <jose.bollo@iot.bzh>
Change-Id: I1baac1b65014e840acb6ecedb501fc14abc17ed6

Fix bug in C++ wrapper

Address of the function is much better than adress of the
pointer in the stack.

BUG-AGL: SPEC-3220

Change-Id: I860b66ac7b8581af5229201556333edcde1f089f
Signed-off-by: Jose Bollo <jose.bollo@iot.bzh>

Fix a tiny issue in the protocol

The id sent was 16 bits in one case and 32 bits in the other

Bug-AGL: SPEC-3220

Change-Id: If38c99b2d903b6851ff4682e1f8d771aec7ddcee
Signed-off-by: Jose Bollo <jose.bollo@iot.bzh>

Fix bug in session and token removal

The bug is trivial and was revealed by testing.
A copy paste bug...

Bug-AGL: SPEC-3220

Change-Id: I65ed8a3a3e377338fd41a2b64b8698f421bbacb9
Signed-off-by: Jose Bollo <jose.bollo@iot.bzh>

Improves compatibility with AddressSanitizers

Address sanitizers is expecting shared objects loaded
without the flag RTLD_DEEPBIND. This can be achieved
by setting the environment variable AFB_NO_RTLD_DEEPBIND
to the value 1 (exactly).

This implies that the binder takes care of not clashing
the namespaces by defining the symbol afbBindingV3root
in both environment: the binder and the bindings.

Bug-AGL: SPEC-3040
Bug-AGL: SPEC-3162

Signed-off-by: José Bollo <jose.bollo@iot.bzh>
Change-Id: Ic064590178d4cf8b34b939e4c9ecd587668f71b8

Update .gitreview file

This updates the gitreview file in the project

.

Change-Id: I74fa71680335cff27e3bf42518026c09ea47f527
Signed-off-by: Jan-Simon Möller <jsmoeller@linuxfoundation.org>

Fix globset compilation warning

The case of returning NULL is not tested in `globset_add`.
It is safe because `grow` is called first and then `search`
never returns a NULL pointer.

Bug-AGL: SPEC-3134

Signed-off-by: José Bollo <jose.bollo@iot.bzh>
Change-Id: I21e8ba3a6c8202a920c6784f4d61376450b9a063

Update copyright dates

Signed-off-by: José Bollo <jose.bollo@iot.bzh>
Change-Id: I01a566a693b8ec6239209c9323ae4ff15a07f737

afb-ws-client: Improve direct URI

The direct connection URI was expecting a slash after the port
for "tcp:" URI.  This slash is not needed because it does not
carry any meaning. This change implements the ability to set
or not the path of URI.

Bug-AGL: SPEC-3094

Change-Id: I3d4c350f4e917759cbf82ccc0d261aa420f6f88a
Signed-off-by: Jose Bollo <jose.bollo@iot.bzh>

afb-evt: Ensure unsubscribe works

Fix the logic of unsubscribing to events. It
was not possible before to implment it without
tracking every session and context. It was not
done because of the required complexity.

This implementation ensures that unexpected
events lead to a removal of the listener from the
list of watchers of the events.

The management of the list of watchers is reworked
to free unused memory.

Bug-AGL: SPEC-3069

Change-Id: Ie67372adbde9dcb9dc6c5c2738111d22609e7256
Signed-off-by: Jose Bollo <jose.bollo@iot.bzh>

afb-evt: Improve name of listening functions

Some more improvement on the road of the following bug:

Bug-AGL: SPEC-3069

Signed-off-by: José Bollo <jose.bollo@iot.bzh>
Change-Id: I5ad6439fa6051566ab2caf271345eda1efcef398

afb-proto-ws: Add message for unexpected event

This new message allows to revoke an event no more
expected or listened.

Bug-AGL: SPEC-3069

Change-Id: I71945e322276f29b01b628bdf43d75599b521fc3
Signed-off-by: Jose Bollo <jose.bollo@iot.bzh>

afb-proto-ws: Improve comment and names

Some more comment and naming improvement on the road
of the following bug:

Bug-AGL: SPEC-3069

Change-Id: I001ccfba2ea59604d217feb6afe3e5a313f3fbaf
Signed-off-by: Jose Bollo <jose.bollo@iot.bzh>

AFB.js: pass event name

The event name is given at second position,
preserving existing code and making its
use optionnal.

Bug-AGL: SPEC-3069

Signed-off-by: José Bollo <jose.bollo@iot.bzh>
Change-Id: Ieeed1762977b990c139d56c4d78e2b539d6927c6

afb-apiset: Fix start of apis

When using dynamically created/destroyed API, the binder
might miss to start some apis. This fixes that issue by
enforcing to restart from beginning until all is initialised.

Bug-AGL: SPEC-3087

Change-Id: I4dd8c60c8465419a46132bf9f6deffa11e172e2a
Signed-off-by: Jose Bollo <jose.bollo@iot.bzh>

main-afb-client-demo: Fix typo

New line markers were missing.

Bug-AGL: SPEC-3066

Change-Id: I4f1cf990edfb7772480ba658b6eda94bdd7e0289
Signed-off-by: Jose Bollo <jose.bollo@iot.bzh>

afb-token: Fix a critical bug

Management of tokens had a big bug, due to insufficent testing.
This fixes the issue that leaded to memory crashes.

BUG-AGL: SPEC-3066

Change-Id: If967ec58ed04dc715d255a5e7c2196133ce3ec4a
Signed-off-by: Jose Bollo <jose.bollo@iot.bzh>

afb-daemon: Fix optionnal port

Since:
 - introduction of interface to set the
   listening port(s) and address(es)
 - use of nss-localuser for discriminating
   binders

there is no more strict need for exporting
the port and substituting it. So the
substitution is make only if needed.

Since removal of token generation, there is no
more need to propoagate any token.

The two possibilities are kept for being used
whene needed and to ensure sweet transition.

Bug-AGL: SPEC-2968

Signed-off-by: José Bollo <jose.bollo@iot.bzh>
Change-Id: I1d2b4d765f244e46de3071baddd8b3d0817b9436

main-afb-daemon: manage listening interfaces

This change allows to specify interfaces to serve
explicitely.

By default the option --port=PORT lead to serving
interface of specification "tcp:*:PORT". Meaning
that all interfaces are listened.

This is intended to be used with localuser family
of hostnames.

Bug-AGL: SPEC-2968

Signed-off-by: José Bollo <jose.bollo@iot.bzh>
Change-Id: I3fb2a77a5c03dd4c2118ebe3267794f79bfc0262

afb-socket: Add ability to set a default scheme

This is a basic improvement for allowing simple
option setting when uri are to be written.
It allows to set contextually the default scheme.

Bug-AGL: SPEC-2968

Signed-off-by: Jose Bollo <jose.bollo@iot.bzh>
Change-Id: I9d7be96743c47b3dd788a35b0b3642387613bacb

Relax constraint on aliases

Without relaxing constraint, it is impossible to alias
2 directory contents for the same prefix. Relaxing it
is needed for implementing ".well-known" interface.

Bug-AGL: SPEC-2968

Signed-off-by: Jose Bollo <jose.bollo@iot.bzh>
Change-Id: I339699bbe66ed3df74ed4ac92344646339e5f2ad

AFB.js: Enforce single code

AFB.js was present with 2 different versions.
This make it unique.

Bug-AGL: SPEC-2968

Signed-off-by: José Bollo <jose.bollo@iot.bzh>
Change-Id: Iee7a5a08fa1457d572000e42da4147c9fa5944b8

afb-perm: separate access to permission db

Access to permission database is better handled
in a separate file. It will afterward evolve to
integrate cynagora.

Bug-AGL: SPEC-2968

Signed-off-by: José Bollo <jose.bollo@iot.bzh>
Change-Id: Iebcd4e227e3e6c318029926499afb9d41d3f72c7

afb-supervision: Improve messages

The messages prompted when the supervisor is
absent were of to high priority.

Bug-AGL: SPEC-2968

Signed-off-by: José Bollo <jose.bollo@iot.bzh>
Change-Id: Ic53a3039c2d3ed5be0138ec3215d58e27758a79c

afb-context & afb-token: rework token validation

Validation of token is now linked to backend permission
database.

Bug-AGL: SPEC-2968

Signed-off-by: José Bollo <jose.bollo@iot.bzh>
Change-Id: I30b049f92b8324740abecbb9539f7413ad55f7ec

afb-context: Move credentials to context

The split between context and credentials
in requests was somehow artificial and awkward.

This change move the credentials to the context
and removes as many references to credentials
as possible in favor of working on contexts.

Change the value returned by afb_auth_check to be 1
if validated, 0 or less than zero if not validated.

Bug-AGL: SPEC-2968

Signed-off-by: José Bollo <jose.bollo@iot.bzh>
Change-Id: I979dc841e03247e126e3fa8433a1cc0d4108adf0

afb-hswitch: Update websocket policy

The previous policy for websocket was to enforce them
to present a valid token when establishing. This policy
is removed because the Websocket API of javascript doesn't
provide a way to set the Bearer token in the HTTP header
of the negociation (a big miss in my opinion).

But because the new policy of token logic is to check
the token at each request, it is not more needed to check
it at establishment.

Bug-AGL: SPEC-2968

Change-Id: I2941757492a27a2eed14e26fbb411330ab4aa8bc
Signed-off-by: Jose Bollo <jose.bollo@iot.bzh>

afb-auth: Increase and improve use of afb-auth

This change factorize code for version V1 of bindings
and centralizes management of authorisations in a
single place.

Bug-AGL: SPEC-2968

Change-Id: I6ad95d5bfa0d85dbb6d2060fc9ebca08b68eb4e9
Signed-off-by: Jose Bollo <jose.bollo@iot.bzh>

afb-error-text: Introduce standard error text

The standard error text are used to return standard
HTTP error codes.

Bug-AGL: SPEC-2968

Change-Id: Ic70e7982b1e05a1830cfa4e54813227621192ae2
Signed-off-by: Jose Bollo <jose.bollo@iot.bzh>

afb-proto-ws: Change the protocol WSAPI

Change internals of the protocol WSAPI for the following
rationale:

 1. Enforce specific declaration and transmission of session
    identifiers and of access tokens.
 2. Lower the size of identifiers to be 16 bits.
 3. Introduce protocol versionning through a mechanism of offer/set.

The main purpose of that change is to optimize the count of data
transmitted. It manages as best as possible the transmission of
access tokens the less possible times. Same for sessions that
the chage was transmitted at each call.

Bug-AGL: SPEC-2968

Change-Id: If0a22b86627ead35a410e51c1028025c5b02c38f
Signed-off-by: Jose Bollo <jose.bollo@iot.bzh>

afb-stub-ws: Enforce asynchronous describe

Because remote apis describe themselves asynchronousely,
it is better to have asynchronous describe api.

Bug-AGL: SPEC-2968

Change-Id: I52b4dab697f229ad01ea2b73d6b8dee22d507912
Signed-off-by: Jose Bollo <jose.bollo@iot.bzh>

afb-evt: Use 16 bits for ids

16 bits are enough

Bug-AGL: SPEC-2968

Change-Id: I0e8708c1d6a3934a342721a6ce5edb4676df6d1b
Signed-off-by: Jose Bollo <jose.bollo@iot.bzh>

u16id: Add maps for identifiers id of 16 bits

These maps will shortly be used by protocol to handle
tokens/sessions/events.

Bug-AGL: SPEC-2968

Change-Id: Iadef7d6e01c8ef021516749524b10ccc1abec340
Signed-off-by: Jose Bollo <jose.bollo@iot.bzh>

afb-session: remove initial token

Tokens are no more managed by the binder.

Bug-AGL: SPEC-2968

Change-Id: Idf1c38105f8ede82cd0ed43aeae3e8e69b9845a3
Signed-off-by: Jose Bollo <jose.bollo@iot.bzh>

afb-session: Expose a local id for sessions

Also, improve readability and memory footprint

Bug-AGL: SPEC-2968

Change-Id: I90104fb19a44e2810b92bef51d774acf734fd90f
Signed-off-by: Jose Bollo <jose.bollo@iot.bzh>

Use afb_token in contexts

Tokens are now object used in the context.

Bug-AGL: SPEC-2968

Change-Id: I107d31732202b7b1172afaf09f3a52470f050d7c
Signed-off-by: Jose Bollo <jose.bollo@iot.bzh>

Move tokens from sessions to requests

Tokens are no more related to sessions.

Each request provides a token. In the case of websockets or
connected link, the context can record the token.

Bug-AGL: SPEC-2968

Change-Id: I1442b0422584c5a5b860ddb826518b0e673612f9
Signed-off-by: Jose Bollo <jose.bollo@iot.bzh>

Don't return the uuid

uuid is available in cookies and through
a call to monitor/session

Bug-AGL: SPEC-2968

Change-Id: I72912f3dc7985cca09e77c952e416b608711abbe
Signed-off-by: Jose Bollo <jose.bollo@iot.bzh>

Remove returning token

The token is no more generated by the binder

Bug-AGL: SPEC-2968

Change-Id: I43f66c7613b42d33058e167b831a93b2112d3b9e
Signed-off-by: Jose Bollo <jose.bollo@iot.bzh>

Remove refreshing token

The token is no more generated by the binder
but by some external component.

Bug-AGL: SPEC-2968

Change-Id: I2c6221034272ab097e21e7727e4840b6b47bd0dc
Signed-off-by: Jose Bollo <jose.bollo@iot.bzh>

afb-hreq: Handle access_token query parameter

As specified by OAuth2 protocols, the access token can be
passed as a POST/GET parameter of name 'access_token'.

Bug-AGL: SPEC-2968

Signed-off-by: José Bollo <jose.bollo@iot.bzh>
Change-Id: I0e02e6fd0d53dad6de994d4482350fe42ecfce48

monitoring: Fix display on connection error

When the connection can't occur, the output is not correct.

Bug-AGL: SPEC-2975

Change-Id: I9cc185329ddccb00f2544dd7b17841b108cc2c81
Signed-off-by: Jose Bollo <jose.bollo@iot.bzh>

coverage: Fix regression

The coverage test was broken. This restores it.

Also add a simple build script that covers most
of the code by default.

Bug-AGL: SPEC-2974

Change-Id: I4fc5a0ba54195738405460a04f985bb67b81b5df
Signed-off-by: Jose Bollo <jose.bollo@iot.bzh>

Introduce object for tokens

For further optimizations, the token is represented
by a specific structure.

Bug-AGL: SPEC-2968

Change-Id: I3d46a12c8c16809c6cc1d543fa2e6309927ed84d
Signed-off-by: Jose Bollo <jose.bollo@iot.bzh>

afb-hreq: Handle HTTP header Authorization

Allows the client to pass its authorization
token using the standard RFC 6750 method.

Bug-AGL: SPEC-2968

Change-Id: Ie9428f4b63554af121b091282ae2c126b4d0c020
Signed-off-by: Jose Bollo <jose.bollo@iot.bzh>

afb-proto-ws: Fix crash on event to disconnected

There was a race condition that made the binder
crashing when reporting event to a client that was
disconnecting.

Bug-AGL: SPEC-2967

Change-Id: I37a654960b42fbce5548ace9d3fb50cf2b375090
Signed-off-by: Jose Bollo <jose.bollo@iot.bzh>

afb-api-dbus: Fix broken broadcast in dbus apis

Probably because of conditional compiling,
afb-api-dbus was broken since introduction
of uuid and hops in broadcats. This change
fixes this issue.

Bug-AGL: SPEC-2969

Change-Id: I9a0fd8552f5f61d6eafa1ed83f6e09e6474378ff
Signed-off-by: Jose Bollo <jose.bollo@iot.bzh>

docs(func-service.md): typo

Bug-AGL: SPEC-2714

Change-Id: I913a836d8b762771d4a966870569b48e4c06804c
Signed-off-by: Li Xiaoming <lixm.fnst@cn.fujitsu.com>

jobs: Fix callsync hanging

The function implementing jobs_enter, used by
implementations of synchronous calls, was not
taking care of waking up a thread. This had the
effect of blocking calls made by an external thread.

Bug-AGL: SPEC-2937

Change-Id: I4bf0265b4c029fb619ef7128824ee9d46a45996e
Signed-off-by: Jose Bollo <jose.bollo@iot.bzh>

sig-monitor: Fix exit in signal handler

Calling exit in signal interrupts wasn't correctly handling
the case where the signal interrupts a thread waiting in the
main loop. This can lead to the binder error report:

    CRITICAL: Can't enter dispatch while in dispatch!

This patch defers the call to exit in a job.

Bug-AGL: SPEC-2907

Signed-off-by: José Bollo <jose.bollo@iot.bzh>
Change-Id: I49c7cca1d229ae957d9ea9bfb8838161ce73a53e

Improve detection of the bindings

The symbol 'afbBindingV3root' may be defined
in shared libraries that are not bindings.
Using it to detect bindings v3 can lead to
false detection and errors.

Bug-AGL: SPEC-2841

Signed-off-by: José Bollo <jose.bollo@iot.bzh>
Change-Id: Ib2bc77d95e3306e7b4a883136ce27c92f8524a8d

Revert "Make source files relative"

This reverts commit 45a4e69d4ef15c70d2b74ea4d0d2e4cd22e3adea.

Revert looks good because the issue has to be adressed by
the build environment.

Bug-AGL: SPEC-2801

Signed-off-by: José Bollo <jose.bollo@iot.bzh>
Change-Id: Id2f1f3890b5965e0a28f189834c758be5bd08d55

jobs: Wake up an event loop if needed

A queued job must be treated. It was not the case
when the job was queued from a foreign thread.
This change detect that a potential hang exists
and wake up an event loop to avoid it.

Bug-AGL: SPEC-2809

Signed-off-by: José Bollo <jose.bollo@iot.bzh>
Change-Id: Id12d32771ea37df5f5f2e208ec9645a6c4b0d0ab

Make source files relative

It is intended to enhance reproducibility
of produced binaries as unmeaning prefixes
are removed.

Bug-AGL: SPEC-2801

Signed-off-by: José Bollo <jose.bollo@iot.bzh>
Change-Id: I6a7ee9fa2ce4a744f7a5ef8a3c9886cd6a9f15e9

Fix binding example path & update afb-daemon options

Bug-AGL: SPEC-2714

Change-Id: I78aee6b3d96728a7eb394ddd94000c516fbc63ba
Signed-off-by: Li Xiaoming <lixm.fnst@cn.fujitsu.com>

Fix client disconnection close

Before that change, the client disconection made a loop
on read but wasn't closing the link.

Bug-AGL: SPEC-2759

Signed-off-by: José Bollo <jose.bollo@iot.bzh>
Change-Id: Iecfbb95cb1b7079e695ff5c08155b2310bf4b8d8

jobs: Add starting mode for jobs

Also improve naming of variables.

Bug-AGL: SPEC-2720

Change-Id: I8efa74e27256425df18f57b3de257c234ff60e69
Signed-off-by: Jose Bollo <jose.bollo@iot.bzh>

Replace obsolete argument "sessiondir" with "workdir"

afb-daemon can not recognize arg "sessiondir" now, it will report
"Error: Bad option detected".Use "workdir" instead.

Bug-AGL: SPEC-2714

Change-Id: I0a194efa7a37ce3e6624c44e48d3e98dcfdda4c2
Signed-off-by: Li Xiaoming <lixm.fnst@cn.fujitsu.com>

Avoid lock when child dies

Interaction between pthread_cond_wait, pthread_broadcast
and signal may lead to lock when the child of afb-daemon dies.

To avoid that sad behavior, this commit defers the call to
"job_exit" using lazy job queueing.

Bug-AGL: SPEC-2720

Change-Id: Ifd1a56f4a439e1704f79a1291fa01f39b1640f29
Signed-off-by: Jose Bollo <jose.bollo@iot.bzh>

Fix outdated documents

Bug-AGL: SPEC-2714

Change-Id: I198f93184a0389f017c9da5b64e800b6faec8cac
Signed-off-by: Li Xiaoming <lixm.fnst@cn.fujitsu.com>

Fix detection of error in call synchronous

There fixes a bug with an inversion between info
and error in implementation of synchronous calls,
leading to bad detection of errors.

Bug-AGL: SPEC-2697

Change-Id: I30a8cd9c45a91b442f6c501dd3b0c32cd3ea632d
Signed-off-by: Jose Bollo <jose.bollo@iot.bzh>

supervisor: Setup services for local API use

The supervisor only allowed external accesses.
This was wrong, it must also allow local access
through standard API. This commit fix it.

Bug-AGL: SPEC-2660

Signed-off-by: José Bollo <jose.bollo@iot.bzh>
Change-Id: Ifa1119a6b2f22c87b1dbe087206d9f51c4005a57

AFB.js: Improve default token discovery

The example script AFB.js should look int the URLs
of the page if the token is set or not and then use it.

Bug-AGL: SPEC-2661

Signed-off-by: Jose Bollo <jose.bollo@iot.bzh>
Change-Id: I4adba8eac214130b425b6cd3c56fc1dc63543011

afb-binding.h: Ensure json-c/json.h is sourced

This is needed if we want to replace json-c with jsonfast.

Bug-AGL: SPEC-2659

Signed-off-by: Jose Bollo <jose.bollo@iot.bzh>
Change-Id: I32da71bd05bfe3538596cbf751d55ff45b4fe7f9

afb-xreq: Remove field 'listener'

This simplifies the flow that now has only one
case: the callbacks subscribe/unsubscribe of
struct afb_xreq_query_itf.

Bug-AGL: SPEC-2658

Signed-off-by: Jose Bollo <jose.bollo@iot.bzh>
Change-Id: Ia49f3ce7aaa4d8603014fe163f311eadfcbf68e4

Fix bad memory access at client disconnection

The management of structures handling a client
connection to a exported --ws-server was accessing
freed memory. Tha commit fixes that issue.

Bug-AGL: SPEC-2651

Change-Id: I511218afc907308347bc422a8aead32ca00bdae6
Signed-off-by: Jose Bollo <jose.bollo@iot.bzh>

Tag broadcasted events with UUID and hop

When API have mutual dependencies, leading to
loops in dependecies, broadcasting an event
never ends because of the loop (see SPEC-2625).

To avoid that weird flood of events, a unic
identifier (UUID) is attached to broadcasted
event and a tiny memory records previously
broadcasted events to avoid re-sending an
already sent event.

The size of the memory can be set using the
macro variable EVENT_BROADCAST_MEMORY_COUNT
whose default value is 8. It can be reduced
to 0.

An other mecanism is added to limit the count
of hops that a broadcasted event can do. That
count can be set using the macro variable
EVENT_BROADCAST_HOP_MAX whose default value
is 10.

Bug-AGL: SPEC-2625

Change-Id: I29550a8a5c8f5e2ffb20edc3330357a1e870c0e2
Signed-off-by: José Bollo <jose.bollo@iot.bzh>

afb-evt: Refactor processing of broadcasted events

Solving the bug SPEC-2625 needs to rework the broadcasting
of events. It appeared that the numerical event identifier
passed for broadcast wasn't used by called function except
for hooking. Suppressing it introduces a clear distinction
between the push and the broadcast paths.

The file afb-ws-json1 is changed to avoid casting of
functions.

Bug-AGL: SPEC-2625

Change-Id: I9fe75adc8086812b21b70ce28baffcf77bd5e1cf
Signed-off-by: José Bollo <jose.bollo@iot.bzh>

uuid: Add module for UUID generation

This commit allow modules of afb-daemon to request UUID.

This introduce an abstraction above libuuid for 3 reasons:

 1. Using uuid had bad side effect inthe past because it
    required enought entropy at start time, leading to hanging
    initialisations in wait for enough entropy.

 2. Solving bug SPEC-2625 will require use of UUID.

 3. Porting to systems that haven't libuuid is made more
    easy after that.

Bug-AGL: SPEC-2625

Change-Id: I27d8b611946686dea6fab250697afe9b89aac869
Signed-off-by: José Bollo <jose.bollo@iot.bzh>

afb-client-demo: Allow pipe of messages

The tool afb-client-demo implements a synchronous
mode (option -s) that allows to wait for a reply
before to send the next request.

It is interesting for the purpose of testing to allow
afb-client-demo it to send a controlled count of
requests, ensuring that there is a controlled count
of pending requests.

Bug-AGL: SPEC-2638

Signed-off-by: Jose Bollo <jose.bollo@iot.bzh>
Change-Id: Ic854d68aaeb82c17b4640c5c9de46181ad3a7e5e

afb-xreq: Forbids (un)subscribes after reply

Allowing to subscribe to a request that was replied
was possible and lead to problem revealed by issues
SPEC-2542 and SPEC-2599 (these issues are more related
to processing unordered incoming messages).

The choice was
 - fix the bug and authorize (un)subscribe after reply
 - forbids to (un)subscribe after reply

Second solution was chosen for its simplicity and
its networking efficiency.

Bug-AGL: SPEC-2542
Bug-AGL: SPEC-2599

Change-Id: I09f48b760b1fd6f70d42b80df3c8053696c45966
Signed-off-by: Jose Bollo <jose.bollo@iot.bzh>

Revert "afb-stub-ws: Allow unordered process of messages"

This reverts commit 5425e054fbf87fe6d024103f46e53f2a28e074f2.

The change introduced in the reverted commit
is no more necessary since events are sent asynchronousely.

Bug-AGL: SPEC-2215
Bug-AGL: SPEC-2219
Bug-AGL: SPEC-2542
Bug-AGL: SPEC-2599

Signed-off-by: Jose Bollo <jose.bollo@iot.bzh>
Change-Id: I4fd8d1516c97e2f95caaa244e28491f71b15bba7

afb-evt: send events in separate job

Sending events in the context of the calling process
had the consequence that the ordering of the messages
had to be removed (see SPEC-2215 & SPEC-2219). This
was not good by nature and lead to issues SPEC-2542
and SPEC-2599.

Sending events in the context of the calling process
also implies to delay the calling process.

For this reasons, sending events is now done in an
other jobs. For that reason, the count of allowed
pending jobs is increased to 100 (was 50).

Bug-AGL: SPEC-2215
Bug-AGL: SPEC-2219
Bug-AGL: SPEC-2542
Bug-AGL: SPEC-2599

Change-Id: I5b56d952cc187b65ad6eb9344ad74e5e8d3b7540
Signed-off-by: Jose Bollo <jose.bollo@iot.bzh>

afb-evt: Improve compatibility to guppy

The commit "a2cf84e: hooks: Allow to remove hooking"
re-order fragment of code. But this is an issue when
backporting changes to guppy.

That commit improve the situation for the file
afb-evt.c in the process of fixing the issue
SPEC-2599

Bug-AGL: SPEC-2599

Signed-off-by: José Bollo <jose.bollo@iot.bzh>
Change-Id: I4e56cbeee7d3a89770e37a3045f652f39fc07410

afb-args: Fix output of --version

The legacy V1 and V2 versions of bindings are no
more printed if absent.

The support of dynamic bindings is fixed.

Bug-AGL: SPEC-2617

Signed-off-by: José Bollo <jose.bollo@iot.bzh>
Change-Id: I429c0e2cdfcbfba6e0f1e639dbcbc2ca90a345ed

Fix syntax error in a constexpr function

The function declaration contains a syntax error that was somehow
ignored but raise an error about an illegal cast in a constexpr
function.
Fixed this error and clean up a bit surrounding code to be more concise.

Bug-AGL: SPEC-2615

Change-Id: I931b086c96b093b3de4465c51dfc3e865f7ece3b
Signed-off-by: Loïc Collignon <loic.collignon@iot.bzh>

Fix invalid TRACE setting in print version

Bug-AGL: SPEC-2617

Change-Id: Ia935aab3b0b508e7a916e334237c639efb1feb15
Signed-off-by: Sebastien Douheret <sebastien.douheret@iot.bzh>

Fix warning when compiling with clang

The 'maybe-uninitialized' warning option is a GCC's only flag. The
equivalent one for Clang is 'possible-uninitialized'.
Added the right flag based on the compiler.

Bug-AGL: SPEC-2584

Change-Id: I09ce434dfda21996cefbf0b527b43d666a042e0e
Signed-off-by: Loïc Collignon <loic.collignon@iot.bzh>

Build af-binder-devtools-native using GCC-9.1.1

GCC-9.1.1 complains about variable ‘buf[0]’ that it may be used uninitialized
in the function 'make_info()'. This patchset initialize the variable.

Bug-AGL: SPEC-2482
Signed-off-by: Bechir Mghirbi <bechir.mghirbi@grammer.com>
Change-Id: I38d0c6f47fed462f4ea48990c4cb63f21fb2ee16

afb-daemon: Add /tmp has fallback for uploads

During its start, when implementing HTTP server,
afb-daemon checked that the upload directory was
accessible and writable and afb-daemon refused to
start when it was not the case.

When trying to use systemd's DynamicUser for platform
services, it had the effect to forbid service to run.

That commit add the directory /tmp as a fallback
upload directory.

Bug-AGL: SPEC-2446

Signed-off-by: José Bollo <jose.bollo@iot.bzh>
Change-Id: Ib4ee456d71cb1aefa19e8eed6774ef5f77919366

jobs: Fix infinite wait lock

The binder was sometime locked until an external
event comes. This was discovered during stressing
startup test that don't expect external inputs and
that then waits for always without terminating.

The issue came from a little hole in the management
of the state of evmgr object. By design the evmgr
has to be synchronised against concurrent accesses
not by itself but by the integration. However, the
state of "running" wasn't set in the code protected
against concurent accesses. The new function allows
the integrator to correctly set the state before
releasing protection.

Bug-AGL: SPEC-2459

Signed-off-by: José Bollo <jose.bollo@iot.bzh>
Change-Id: Id75fd97c06d69666ced912a0a61380573dab31af

Fix afb document typo.

Bug-AGL: SPEC-2391

Change-Id: I73efab27d09fce7ca16edabe14fe19d1092162db
Signed-off-by: Li Xiaoming <lixm.fnst@cn.fujitsu.com>

afb-export: Fix requirement on multiple apis

The test 'test-apiv3' was failing because the required
api list "albert armel" wasn't splitted but set as-is.

At the same time, test suite is improved.

Bug-AGL: SPEC-1130
Bug-AGL: SPEC-2377

Change-Id: Ia4ab23bf666b5b4b1db3e1da2654d9387c2c01e5
Signed-off-by: José Bollo <jose.bollo@iot.bzh>

coverage: Add test of AFB_DEBUG_WAIT

Bug-AGL: SPEC-1130

Change-Id: I349c141015e3851ddecd4a6df7497d8ef4cceaa6
Signed-off-by: José Bollo <jose.bollo@iot.bzh>

coverage: Allow to run coverage everywhere

Coverage presumed that an available afb-daemon without
coverage instrumentation was available in the PATH.

This fix that problem by compiling that requirement.

Bug-AGL: SPEC-1130

Change-Id: Ic3435d73e830d630bef63dfb8e9b2ff9f1bf043c
Signed-off-by: José Bollo <jose.bollo@iot.bzh>

Allow to tune installation directories

The location of intrinsic bindings and samples could not be tuned
but was fixed to be ${CMAKE_INSTALL_FULL_LIBDIR}/afb and
${CMAKE_INSTALL_FULL_DATADIR}/af-binder

These default locations are not bad but it is important to be able
to tune that location.

Bug-AGL: SPEC-2367

Change-Id: I4d4f9e9490d61e3278ef35ac42f2143a752a7c37
Signed-off-by: José Bollo <jose.bollo@iot.bzh>

doc: Improve comment on asynchronous calls

Following the remarks of fellow developers, the status
of the object received by the callback function
needs to be explained better.

Change-Id: I0e6bcd51849bbc5af92af08675949a901bfd683e
Signed-off-by: Jose Bollo <jose.bollo@iot.bzh>

binding-wrap: Rework of context handling

The C++ wrapper for handling contexts has
to be improved. This is a proposition of
improvement.

Change-Id: I7df36383f427d109356bdf4df573cba4b6e6ec05
Signed-off-by: Jose Bollo <jose.bollo@iot.bzh>

samples: wrong parameter for get_new_board()

for fixing
warning: too many arguments in call to 'get_new_board'

Change-Id: I7b3d175310156ea3d2efed29e344f01c9904d534
Signed-off-by: Phong Tran <tranmanphong@gmail.com>

Merge changes Id24d9bb0,I6807ed25

* changes:
  change style initialization of struct afb_auth
  missing full initialization for struct afb_verb_v3

change style initialization of struct afb_auth

This is for fixing
warning: suggest braces around initialization of subobject
[-Wmissing-braces]

Change-Id: Id24d9bb09e46d59df7d043ed931e00bb5ec9469a
Signed-off-by: Phong Tran <tranmanphong@gmail.com>

missing full initialization for struct afb_verb_v3

This is for fixing
warning: missing field 'callback' initializer
[-Wmissing-field-initializers]

Change-Id: I6807ed25bff96c96f347ec1e87f1b12cf0759498
Signed-off-by: Phong Tran <tranmanphong@gmail.com>

Merge "afb-migration-to-binding-v3: fix broken links"