Merged revisions 98946 via svnmerge from
authorRussell Bryant <russell@russellbryant.com>
Tue, 15 Jan 2008 23:53:28 +0000 (23:53 +0000)
committerRussell Bryant <russell@russellbryant.com>
Tue, 15 Jan 2008 23:53:28 +0000 (23:53 +0000)
commit2cdf636c0fa575ef56506071a4856c964ba9d37a
treedd5c109afac147375ab8527dde6bc016708f63fd
parent799246dae34ebe986b71fb20aa24bf25cbb19778
Merged revisions 98946 via svnmerge from
https://origsvn.digium.com/svn/asterisk/branches/1.4

........
r98946 | russell | 2008-01-15 17:50:10 -0600 (Tue, 15 Jan 2008) | 11 lines

Change a buffer in check_auth() to be a thread local dynamically allocated
buffer, instead of a massive buffer on the stack.  This fixes a crash reported
by Qwell due to running out of stack space when building with LOW_MEMORY defined.

On a very related note, the usage of BUFSIZ in various places in chan_sip is
arbitrary and careless.  BUFSIZ is a system specific define.  On my machine,
it is 8192, but by definition (according to google) could be as small as 256.
So, this buffer in check_auth was 16 kB.  We don't even support SIP messages
larger than 4 kB!  Further usage of this define should be avoided, unless it
is used in the proper context.

........

git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@98948 65c4cc65-6c06-0410-ace0-fbb531ad65f3
channels/chan_sip.c