This commit is changing the keepalive exported api and the modules using it
don't build anymore, for now drouting is reported by LGTM analysis to be broken:
```
[2020-05-04 14:29:31] [build] CC (gcc) [M drouting.so] drouting.o
[2020-05-04 14:29:31] [build] drouting.c: In function ‘dr_update_keepalive’:
[2020-05-04 14:29:31] [build] drouting.c:226:26: warning: passing argument 4 of
‘keepalive_api.add_destination’ makes integer from pointer without a cast
[-Wint-conversion]
[2020-05-04 14:29:31] [build] 226 | &cur->ip, &owner, 0,
dr_keepalive_statechanged, cur);
[2020-05-04 14:29:31] [build] |
^~~~~~~~~~~~~~~~~~~~~~~~~
[2020-05-04 14:29:31] [build] | |
[2020-05-04 14:29:31] [build] | void (*)(str *,
ka_state, void *) {aka void (*)(struct _str *, int, void *)}
[2020-05-04 14:29:31] [build] drouting.c:226:26: note: expected ‘int’ but
argument is of type ‘void (*)(str *, ka_state, void *)’ {aka ‘void (*)(struct
_str *, int, void *)’}
[2020-05-04 14:29:31] [build] drouting.c:226:53: warning: passing argument 5 of
‘keepalive_api.add_destination’ from incompatible pointer type
[-Wincompatible-pointer-types]
[2020-05-04 14:29:31] [build] 226 | &cur->ip, &owner, 0,
dr_keepalive_statechanged, cur);
[2020-05-04 14:29:31] [build] |
^~~
[2020-05-04 14:29:31] [build] |
|
[2020-05-04 14:29:31] [build] |
pgw_t * {aka struct pgw_ *}
[2020-05-04 14:29:31] [build] drouting.c:226:53: note: expected
‘ka_statechanged_f’ {aka ‘void (*)(struct _str *, int, void *)’} but argument
is of type ‘pgw_t *’ {aka ‘struct pgw_ *’}
[2020-05-04 14:29:31] [build] drouting.c:225:3: error: too few arguments to
function ‘keepalive_api.add_destination’
[2020-05-04 14:29:31] [build] 225 | keepalive_api.add_destination(
[2020-05-04 14:29:31] [build] | ^~~~~~~~~~~~~
[2020-05-04 14:29:32] [build] make[2]: *** [../../Makefile.rules:100:
drouting.o] Error 1
[2020-05-04 14:29:32] [build] make[1]: *** [Makefile:511: modules] Error 1
[2020-05-04 14:29:32] [build] make[1]: Leaving directory '/opt/src/src'
[2020-05-04 14:29:32] [build] make: *** [Makefile:34: all] Error 2
```
--
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/kamailio/kamailio/pull/2313#issuecomment-623874760_______________________________________________
Kamailio (SER) - Development Mailing List
sr-dev@lists.kamailio.org
https://lists.kamailio.org/cgi-bin/mailman/listinfo/sr-dev
