[asterisk-dev] Re: [asterisk-commits] oej: branch 1.4 r48166 - in /branches/1.4: ./ channels/chan_sip.c

[Luigi Rizzo]

On Thu, Nov 30, 2006 at 08:51:04PM -0000, asterisk-commits at lists.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=48166
> 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/channels/chan_sip.c?view=diff&rev=48166&r1=48165&r2=48166
> ==============================================================================
> --- 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-commits