Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

ims_registrar_scscf: cannot build on Fedora 40 #3855

Open
sergey-safarov opened this issue May 16, 2024 · 1 comment
Open

ims_registrar_scscf: cannot build on Fedora 40 #3855

sergey-safarov opened this issue May 16, 2024 · 1 comment

Comments

@sergey-safarov
Copy link
Member

Description

"ims_registrar_scscf" cannot be compiled on Fedora 40 dist.

CC (gcc) [M ims_registrar_scscf.so]		reply.o
CC (gcc) [M ims_registrar_scscf.so]		rerrno.o
CC (gcc) [M ims_registrar_scscf.so]		save.o
CC (gcc) [M ims_registrar_scscf.so]		server_assignment.o
CC (gcc) [M ims_registrar_scscf.so]		sip_msg.o
CC (gcc) [M ims_registrar_scscf.so]		stats.o
CC (gcc) [M ims_registrar_scscf.so]		userdata_parser.o
userdata_parser.c: In function ‘ifc_tDefaultHandling2char’:
userdata_parser.c:126:13: error: implicit declaration of function ‘strtol’; did you mean ‘strtok’? [-Wimplicit-function-declaration]
  126 |         r = strtol((char *)x, (char **)NULL, 10);
      |             ^~~~~~
      |             strtok
userdata_parser.c: In function ‘parse_spt_extension’:
userdata_parser.c:411:32: error: implicit declaration of function ‘atoi’ [-Wimplicit-function-declaration]
  411 |                         switch(atoi((char *)x)) {
      |                                ^~~~
make[2]: *** [../../Makefile.rules:100: userdata_parser.o] Error 1
make[1]: *** [Makefile:508: modules] Error 1
make[1]: Leaving directory '/root/rpmbuild/BUILD/kamailio-5.8.1/src'
make: *** [Makefile:34: every-module] Error 2
error: Bad exit status from /var/tmp/rpm-tmp.tyeQgK (%build)

RPM build errors:
    Bad exit status from /var/tmp/rpm-tmp.tyeQgK (%build)

Additional Information

  • Kamailio Version - output of kamailio -v
used modified 5.8.1 version
  • Operating System:
[root@sbc-stage-a0 SPECS]# cat /etc/os-release 
NAME="Fedora Linux"
VERSION="40 (Container Image)"
ID=fedora
VERSION_ID=40
VERSION_CODENAME=""
PLATFORM_ID="platform:f40"
PRETTY_NAME="Fedora Linux 40 (Container Image)"
ANSI_COLOR="0;38;2;60;110;180"
LOGO=fedora-logo-icon
CPE_NAME="cpe:/o:fedoraproject:fedora:40"
DEFAULT_HOSTNAME="fedora"
HOME_URL="https://fedoraproject.org/"
DOCUMENTATION_URL="https://docs.fedoraproject.org/en-US/fedora/f40/system-administrators-guide/"
SUPPORT_URL="https://ask.fedoraproject.org/"
BUG_REPORT_URL="https://bugzilla.redhat.com/"
REDHAT_BUGZILLA_PRODUCT="Fedora"
REDHAT_BUGZILLA_PRODUCT_VERSION=40
REDHAT_SUPPORT_PRODUCT="Fedora"
REDHAT_SUPPORT_PRODUCT_VERSION=40
SUPPORT_END=2025-05-13
VARIANT="Container Image"
VARIANT_ID=container
@sergey-safarov
Copy link
Member Author

This change allows build module

diff --git a/src/modules/ims_registrar_scscf/userdata_parser.c b/src/modules/ims_registrar_scscf/userdata_parser.c
index 984395f886..df629548c6 100644
--- a/src/modules/ims_registrar_scscf/userdata_parser.c
+++ b/src/modules/ims_registrar_scscf/userdata_parser.c
@@ -40,6 +40,7 @@
  *
  */
 
+#include <stdlib.h>
 #include "../../core/parser/parse_uri.h"
 #include "userdata_parser.h"
 #include "../../core/parser/parse_hname2.h"

miconda added a commit that referenced this issue Jun 8, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant