Add guard against NULL args.payload

wdoekes / asterisk-chan-dongle

chan_dongle channel driver for Huawei UMTS cards, works with Asterisk 14+

Other

298 stars 104 forks source link

Closed phcoder closed 4 years ago

phcoder commented 4 years ago

NULL args.payload causes a crash in the call to strlen(args.payload) while computing arguments for send_sms.

wdoekes commented 4 years ago

Looks good. But you might as well use ast_strlen_zero( here too.

Also, please use no space between function and parenthesis (yes, the code above doesn't use the proper convention): ast_log(

Please fix that up, squash and force-push.

phcoder commented 4 years ago

Updated