Asterisk
  1. Asterisk
  2. ASTERISK-12768

[patch] Asterisk crash getting fax by sip channel

    Details

    • Type: Bug Bug
    • Status: Closed
    • Severity: Critical Critical
    • Resolution: Fixed
    • Affects Version/s: None
    • Target Release Version/s: None
    • Labels:
      None
    • Mantis ID:
      13547
    • Regression:
      No

      Description

      Function: sip_uri_headers_cmp

      1. chan_sip.c.gz
        204 kB
        german aracil boned
      2. crash.txt.gz
        8 kB

        Activity

        Hide
        Digium Subversion added a comment -

        Repository: asterisk
        Revision: 174082

        U branches/1.4/channels/chan_sip.c

        ------------------------------------------------------------------------
        r174082 | dhubbard | 2009-02-06 17:36:03 -0600 (Fri, 06 Feb 2009) | 8 lines

        check ast_strlen_zero() before calling ast_strdupa() in sip_uri_headers_cmp() and sip_uri_params_cmp()

        (closes issue ASTERISK-12768)
        Reported by: tecnoxarxa
        Patches:
        chan_sip.c.gz uploaded by tecnoxarxa (license 258)
        Tested by: tecnoxarxa

        ------------------------------------------------------------------------

        http://svn.digium.com/view/asterisk?view=rev&revision=174082

        Show
        Digium Subversion added a comment - Repository: asterisk Revision: 174082 U branches/1.4/channels/chan_sip.c ------------------------------------------------------------------------ r174082 | dhubbard | 2009-02-06 17:36:03 -0600 (Fri, 06 Feb 2009) | 8 lines check ast_strlen_zero() before calling ast_strdupa() in sip_uri_headers_cmp() and sip_uri_params_cmp() (closes issue ASTERISK-12768 ) Reported by: tecnoxarxa Patches: chan_sip.c.gz uploaded by tecnoxarxa (license 258) Tested by: tecnoxarxa ------------------------------------------------------------------------ http://svn.digium.com/view/asterisk?view=rev&revision=174082
        Hide
        Digium Subversion added a comment -

        Repository: asterisk
        Revision: 174082

        U branches/1.4/channels/chan_sip.c

        ------------------------------------------------------------------------
        r174082 | dhubbard | 2009-02-06 17:36:03 -0600 (Fri, 06 Feb 2009) | 14 lines

        check ast_strlen_zero() before calling ast_strdupa() in sip_uri_headers_cmp()
        and sip_uri_params_cmp()

        The reporter didn't actually upload a properly-formed patch, instead a
        modified chan_sip.c file was uploaded. I created a patch to determine the
        changes, then modified the suggested changes to create a proper fix. The
        summary above is a complete description of the changes.

        (closes issue ASTERISK-12768)
        Reported by: tecnoxarxa
        Patches:
        chan_sip.c.gz uploaded by tecnoxarxa (license 258)
        Tested by: tecnoxarxa

        ------------------------------------------------------------------------

        http://svn.digium.com/view/asterisk?view=rev&revision=174082

        Show
        Digium Subversion added a comment - Repository: asterisk Revision: 174082 U branches/1.4/channels/chan_sip.c ------------------------------------------------------------------------ r174082 | dhubbard | 2009-02-06 17:36:03 -0600 (Fri, 06 Feb 2009) | 14 lines check ast_strlen_zero() before calling ast_strdupa() in sip_uri_headers_cmp() and sip_uri_params_cmp() The reporter didn't actually upload a properly-formed patch, instead a modified chan_sip.c file was uploaded. I created a patch to determine the changes, then modified the suggested changes to create a proper fix. The summary above is a complete description of the changes. (closes issue ASTERISK-12768 ) Reported by: tecnoxarxa Patches: chan_sip.c.gz uploaded by tecnoxarxa (license 258) Tested by: tecnoxarxa ------------------------------------------------------------------------ http://svn.digium.com/view/asterisk?view=rev&revision=174082
        Hide
        Digium Subversion added a comment -

        Repository: asterisk
        Revision: 174084

        _U trunk/
        U trunk/channels/chan_sip.c

        ------------------------------------------------------------------------
        r174084 | dhubbard | 2009-02-06 17:51:56 -0600 (Fri, 06 Feb 2009) | 22 lines

        Merged revisions 174082 via svnmerge from
        https://origsvn.digium.com/svn/asterisk/branches/1.4

        ........
        r174082 | dhubbard | 2009-02-06 17:36:03 -0600 (Fri, 06 Feb 2009) | 5 lines

        check ast_strlen_zero() before calling ast_strdupa() in sip_uri_headers_cmp()
        and sip_uri_params_cmp()

        The reporter didn't actually upload a properly-formed patch, instead a
        modified chan_sip.c file was uploaded. I created a patch to determine the
        changes, then modified the suggested changes to create a proper fix. The
        summary above is a complete description of the changes.

        (closes issue ASTERISK-12768)
        Reported by: tecnoxarxa
        Patches:
        chan_sip.c.gz uploaded by tecnoxarxa (license 258)
        Tested by: tecnoxarxa

        ........

        ------------------------------------------------------------------------

        http://svn.digium.com/view/asterisk?view=rev&revision=174084

        Show
        Digium Subversion added a comment - Repository: asterisk Revision: 174084 _U trunk/ U trunk/channels/chan_sip.c ------------------------------------------------------------------------ r174084 | dhubbard | 2009-02-06 17:51:56 -0600 (Fri, 06 Feb 2009) | 22 lines Merged revisions 174082 via svnmerge from https://origsvn.digium.com/svn/asterisk/branches/1.4 ........ r174082 | dhubbard | 2009-02-06 17:36:03 -0600 (Fri, 06 Feb 2009) | 5 lines check ast_strlen_zero() before calling ast_strdupa() in sip_uri_headers_cmp() and sip_uri_params_cmp() The reporter didn't actually upload a properly-formed patch, instead a modified chan_sip.c file was uploaded. I created a patch to determine the changes, then modified the suggested changes to create a proper fix. The summary above is a complete description of the changes. (closes issue ASTERISK-12768 ) Reported by: tecnoxarxa Patches: chan_sip.c.gz uploaded by tecnoxarxa (license 258) Tested by: tecnoxarxa ........ ------------------------------------------------------------------------ http://svn.digium.com/view/asterisk?view=rev&revision=174084
        Hide
        Digium Subversion added a comment -

        Repository: asterisk
        Revision: 174085

        _U branches/1.6.0/
        U branches/1.6.0/channels/chan_sip.c

        ------------------------------------------------------------------------
        r174085 | dhubbard | 2009-02-06 17:59:42 -0600 (Fri, 06 Feb 2009) | 32 lines

        Merged revisions 174084 via svnmerge from
        https://origsvn.digium.com/svn/asterisk/trunk

        ................
        r174084 | dhubbard | 2009-02-06 17:51:56 -0600 (Fri, 06 Feb 2009) | 13 lines

        Merged revisions 174082 via svnmerge from
        https://origsvn.digium.com/svn/asterisk/branches/1.4

        ........
        r174082 | dhubbard | 2009-02-06 17:36:03 -0600 (Fri, 06 Feb 2009) | 5 lines

        check ast_strlen_zero() before calling ast_strdupa() in sip_uri_headers_cmp()
        and sip_uri_params_cmp()

        The reporter didn't actually upload a properly-formed patch, instead a
        modified chan_sip.c file was uploaded. I created a patch to determine the
        changes, then modified the suggested changes to create a proper fix. The
        summary above is a complete description of the changes.

        (closes issue ASTERISK-12768)
        Reported by: tecnoxarxa
        Patches:
        chan_sip.c.gz uploaded by tecnoxarxa (license 258)
        Tested by: tecnoxarxa

        ........

        ................

        ------------------------------------------------------------------------

        ------------------------------------------------------------------------

        http://svn.digium.com/view/asterisk?view=rev&revision=174085

        Show
        Digium Subversion added a comment - Repository: asterisk Revision: 174085 _U branches/1.6.0/ U branches/1.6.0/channels/chan_sip.c ------------------------------------------------------------------------ r174085 | dhubbard | 2009-02-06 17:59:42 -0600 (Fri, 06 Feb 2009) | 32 lines Merged revisions 174084 via svnmerge from https://origsvn.digium.com/svn/asterisk/trunk ................ r174084 | dhubbard | 2009-02-06 17:51:56 -0600 (Fri, 06 Feb 2009) | 13 lines Merged revisions 174082 via svnmerge from https://origsvn.digium.com/svn/asterisk/branches/1.4 ........ r174082 | dhubbard | 2009-02-06 17:36:03 -0600 (Fri, 06 Feb 2009) | 5 lines check ast_strlen_zero() before calling ast_strdupa() in sip_uri_headers_cmp() and sip_uri_params_cmp() The reporter didn't actually upload a properly-formed patch, instead a modified chan_sip.c file was uploaded. I created a patch to determine the changes, then modified the suggested changes to create a proper fix. The summary above is a complete description of the changes. (closes issue ASTERISK-12768 ) Reported by: tecnoxarxa Patches: chan_sip.c.gz uploaded by tecnoxarxa (license 258) Tested by: tecnoxarxa ........ ................ ------------------------------------------------------------------------ ------------------------------------------------------------------------ http://svn.digium.com/view/asterisk?view=rev&revision=174085
        Hide
        Digium Subversion added a comment -

        Repository: asterisk
        Revision: 174086

        _U branches/1.6.1/
        U branches/1.6.1/channels/chan_sip.c

        ------------------------------------------------------------------------
        r174086 | dhubbard | 2009-02-06 18:09:08 -0600 (Fri, 06 Feb 2009) | 30 lines

        Merged revisions 174084 via svnmerge from
        https://origsvn.digium.com/svn/asterisk/trunk

        ................
        r174084 | dhubbard | 2009-02-06 17:51:56 -0600 (Fri, 06 Feb 2009) | 13 lines

        Merged revisions 174082 via svnmerge from
        https://origsvn.digium.com/svn/asterisk/branches/1.4

        ........
        r174082 | dhubbard | 2009-02-06 17:36:03 -0600 (Fri, 06 Feb 2009) | 5 lines

        check ast_strlen_zero() before calling ast_strdupa() in sip_uri_headers_cmp()
        and sip_uri_params_cmp()

        The reporter didn't actually upload a properly-formed patch, instead a
        modified chan_sip.c file was uploaded. I created a patch to determine the
        changes, then modified the suggested changes to create a proper fix. The
        summary above is a complete description of the changes.

        (closes issue ASTERISK-12768)
        Reported by: tecnoxarxa
        Patches:
        chan_sip.c.gz uploaded by tecnoxarxa (license 258)
        Tested by: tecnoxarxa

        ........

        ................

        ------------------------------------------------------------------------

        http://svn.digium.com/view/asterisk?view=rev&revision=174086

        Show
        Digium Subversion added a comment - Repository: asterisk Revision: 174086 _U branches/1.6.1/ U branches/1.6.1/channels/chan_sip.c ------------------------------------------------------------------------ r174086 | dhubbard | 2009-02-06 18:09:08 -0600 (Fri, 06 Feb 2009) | 30 lines Merged revisions 174084 via svnmerge from https://origsvn.digium.com/svn/asterisk/trunk ................ r174084 | dhubbard | 2009-02-06 17:51:56 -0600 (Fri, 06 Feb 2009) | 13 lines Merged revisions 174082 via svnmerge from https://origsvn.digium.com/svn/asterisk/branches/1.4 ........ r174082 | dhubbard | 2009-02-06 17:36:03 -0600 (Fri, 06 Feb 2009) | 5 lines check ast_strlen_zero() before calling ast_strdupa() in sip_uri_headers_cmp() and sip_uri_params_cmp() The reporter didn't actually upload a properly-formed patch, instead a modified chan_sip.c file was uploaded. I created a patch to determine the changes, then modified the suggested changes to create a proper fix. The summary above is a complete description of the changes. (closes issue ASTERISK-12768 ) Reported by: tecnoxarxa Patches: chan_sip.c.gz uploaded by tecnoxarxa (license 258) Tested by: tecnoxarxa ........ ................ ------------------------------------------------------------------------ http://svn.digium.com/view/asterisk?view=rev&revision=174086

          People

          • Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development