[asterisk-dev] [Code Review] 3745: func_uri: URIENCODE/URIDECODE - Remove warning messages when empty string is passed for the argument

Jonathan Rose reviewboard at asterisk.org
Mon Jul 14 11:48:02 CDT 2014


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviewboard.asterisk.org/r/3745/
-----------------------------------------------------------

(Updated July 14, 2014, 11:48 a.m.)


Review request for Asterisk Developers and Matt Jordan.


Changes
-------

Hit rmudgett's finding


Bugs: ASTERISK-23911
    https://issues.asterisk.org/jira/browse/ASTERISK-23911


Repository: Asterisk


Description
-------

WARNING[4943]: func_uri.c:79 uriencode: Syntax: URIENCODE(<data>) - missing argument!

There is nothing especially wrong with trying to encode an empty string, so there is no need to warn for it.
Plus avoiding the warning requires making some dialplans more elaborate.  This:

same => n,Stasis(myapp,${URIENCODE(${ARGS}))

becomes this:

same => n,GotoIf($[${LEN(${ARGS})} = 0]skip:)
same => n,Set(ARGS=${URIENCODE(${ARGS})})
same => n(skip),NoOp()
same => n,Stasis(MyApp,${ARGS})

for instance.


Diffs (updated)
-----

  /branches/1.8/funcs/func_uri.c 418366 

Diff: https://reviewboard.asterisk.org/r/3745/diff/


Testing
-------

Checked to make sure the warning doesn't get generated anymore.  It doesn't.


Thanks,

Jonathan Rose

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-dev/attachments/20140714/5008be4a/attachment.html>


More information about the asterisk-dev mailing list