一个FreeSWITCH Bug修复手记
好久不写技术文章了，今天走一个。
今天有人在微信群里提了一个问题：“杜老师，想麻烦您看一下，电话打进去之后就会出现崩溃，卡了两天了，麻烦您有时间可否帮忙看一下，多谢多谢了，加您好友了 Image。”
看到这条消息的时候，我正好在排查一个公司客户的问题，虽然没有彻底解决，但已定位到一些关键点，正好可以换换脑子，就帮他看了一下。Back Trace 如下：
[Thread debugging using libthread_db enabled]
Using host libthread_db library "/lib64/libthread_db.so.1".
Core was generated by `./freeswitch'.
Program terminated with signal 11, Segmentation fault.
#0 0x00007f7e79c303b1 in __strcasecmp_l_avx () from /lib64/libc.so.6
(gdb) where
#0 0x00007f7e79c303b1 in __strcasecmp_l_avx () from /lib64/libc.so.6
#1 0x00007f7e64f46813 in sofia_reg_parse_auth (profile=profile@entry=0x1707bd0, authorization=authorization@entry=0x7f7e2c03e590,
sip=sip@entry=0x7f7e2c03d1f8, de=de@entry=0x7f7e38014990, regstr=0x7f7e7b3991d0 <sip_method_name_invite> "INVITE",
np=np@entry=0x7f7de99f4e30 "dc6885f2-39b0-4d3d-970a-1b197a16d371", nplen=nplen@entry=128, ip=ip@entry=0x7f7de99f4190 "10.2.18.244",
network_port=58257, v_event=v_event@entry=0x7f7de99f4d58, exptime=exptime@entry=300, regtype=regtype@entry=REG_INVITE,
to_user=to_user@entry=0x7f7e2c03dbc8 "7788", auth_params=auth_params@entry=0x7f7de99f4130, reg_count=reg_count@entry=0x7f7de99f4138,
user_xml=user_xml@entry=0x7f7de99f4d60) at sofia_reg.c:3427
粗看 Back Trace 也没看出具体问题，虽然看到问题大约出在 sofia_reg.c:3427 行，但也不想花时间去翻代码，然后就问了以下几个问题：
- FreeSWITCH 什么版本
- 什么操作系统？
答复是：
- 1.10.7
- CentOS 7.9
看着版本还比较新，就翻了一下代码，如下：
3420 if (input2) {
3421 if (sofia_make_digest(use_alg, &bigdigest, (void *)input2, &digest_outputlen) != SWITCH_STATUS_SUCCESS) {
3422 switch_safe_free(input2);
3423 goto end;
3424 }
3425 }
3426
3427 if (input2 && !strcasecmp(bigdigest, response)) {
初步判断问题出在 3421 行，bigdigest 指针有问题。接着，该同学又提供了 FreeSWITCH 崩溃前的日志如下：
[ERR] switch_utils.c:4584 Unknown message digest md5
[ERR] switch_utils.c:4584 Unknown message digest md5
[ERR] switch_utils.c:4584 Unknown message digest md5
这个就比较明确了，找到代码，发现这里返回了 SWITCH_STATUS_FALSE：
4581 md = EVP_get_digestbyname(digest_name);
4582
4583 if (!md) {
4584 switch_log_printf(SWITCH_CHANNEL_LOG, SWITCH_LOG_ERROR, "Unknown message digest %s\n", digest_name);
4584 return SWITCH_STATUS_FALSE;
再对比 sofia_make_digest 函数，发现该函数调用了 switch_digest_string，进而调用了 switch_digest，但是没有处理返回值，因而导致后续判断不准确造成崩溃。
修复的方法显而易见，参见这里
当然，理论上讲这个函数是不可能返回 FALSE 的，进一步交流发现它用的 OpenSSL 库版本比较低，不知道什么原因导致那个 md5 相关的函数未正常执行，把这个问题留给他当课后作业自己去研究了，本次修改仅是修复了崩溃问题，但他的电话应该还不能正常打。但他在这个问题上已经折腾两天了，相信看到曙光的那一刻心情会大好。大家遇到问题也可以到微信群、QQ 群、知识星球等上面提问，如果我不是特别忙的话，一般是会回答的。
其实这个问题不算复杂。我以前也经常引用高人的话：
如果一个问题被描述的足够详细，那么解决方法是显而易见的。
所以，遇到问题不要慌，以前左耳朵耗子老师讲过一个“橡皮鸭程序调试法”（感兴趣的话自己搜下吧），也是这个道理。
顺便做个广告：欢迎大家试用我们的 XSwitch，有图形界面，简单好用。点击直达。