List Info

Thread: oej: branch 1.4 r48166 - in /branches/1.4: ./ channels/chan_sip.c




oej: branch 1.4 r48166 - in /branches/1.4: ./ channels/chan_sip.c
user name
2006-11-30 20:57:10
On Thu, Nov 30, 2006 at 08:51:04PM -0000,
asterisk-commitslists.digium.com wrote:
> Author: oej
> Date: Thu Nov 30 14:51:04 2006
> New Revision: 48166
> 
> URL: http://svn.digium.com/view/asterisk?view=rev&rev=4
8166
> Log:
> Issue 8319 - change noncecount before using it.

the preincrement operator would do perfectly here

	p->noncecount = ++r->noncecount;

(not that it matters, but in case you need the same feature
elsewhere...)

cheers
luigi

> Modified:
>     branches/1.4/   (props changed)
>     branches/1.4/channels/chan_sip.c
> 
> Propchange: branches/1.4/
>
------------------------------------------------------------
------------------
> Binary property 'branch-1.2-merged' - no diff
available.
> 
> Modified: branches/1.4/channels/chan_sip.c
> URL: http://svn.digium.com/view/asterisk/branches/1.4/chann
els/chan_sip.c?view=diff&rev=48166&r1=48165&r2=4
8166
>
============================================================
==================
> --- branches/1.4/channels/chan_sip.c (original)
> +++ branches/1.4/channels/chan_sip.c Thu Nov 30
14:51:04 2006
>  -7228,7 +7228,8 
>  		ast_string_field_set(p, domain, r->domain);
>  		ast_string_field_set(p, opaque, r->opaque);
>  		ast_string_field_set(p, qop, r->qop);
> -		p->noncecount = r->noncecount++;
> +		r->noncecount++;
> +		p->noncecount = r->noncecount;
>  
>  		memset(digest,0,sizeof(digest));
>  		if(!build_reply_digest(p, sipmethod, digest,
sizeof(digest)))
> 
> _______________________________________________
> --Bandwidth and Colocation provided by Easynews.com --
> 
> asterisk-commits mailing list
> To UNSUBSCRIBE or update options visit:
>    http://lists.digium.com/mailman/listinfo/asterisk-commi
ts
_______________________________________________
--Bandwidth and Colocation provided by Easynews.com --

asterisk-dev mailing list
To UNSUBSCRIBE or update options visit:
   http://lists.digium.com/mailman/listinfo/asterisk-dev
[1]

about | contact  Other archives ( Real Estate discussion Medical topics )