Patch Set 1:


Patch Set 2 has also restored the ability for an fsm instance to have NULL 
passed as an id. I'm not sure if that was used before, but this way the 
behaviour is unchanged.
File include/osmocom/core/fsm.h:

Line 90:        char *id;
> hm. We cannot change a "const char *" pointer?  I always thought it states 
The issue here is the talloc_free() parameter being non-const.

fsm.c:214:16: warning: passing argument 1 of ‘_talloc_free’ discards ‘const’ 
qualifier from pointer target type [-Wdiscarded-qualifiers]

I'll cast the argument to a void* in that call.
File src/fsm.c:

Line 205: bool osmo_fsm_inst_update_id(struct osmo_fsm_inst *fi, const char *id)
> in general we return 0 on success and negative on error.  Only predicate-ty

To view, visit
To unsubscribe, visit

Gerrit-MessageType: comment
Gerrit-Change-Id: Ic216e5b11d4440f8e106a297714f4f06c1152945
Gerrit-PatchSet: 1
Gerrit-Project: libosmocore
Gerrit-Branch: master
Gerrit-Owner: daniel <>
Gerrit-Reviewer: Harald Welte <>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: daniel <>
Gerrit-HasComments: Yes

Reply via email to