Ethereal-dev: Re: [Ethereal-dev] packet-sccp.c patch for compilation under Win32

Note: This archive is from the project's previous web site, ethereal.com. This list is no longer active.

From: Motonori Shindo <mshindo@xxxxxxxxxxx>
Date: Thu, 07 Mar 2002 07:27:25 +0900 (JST)
All,

Sorry, I should have been more thoughtful. Maybe I was too sleepy:-)

Please forget about my patch and apply whatever necessary. Thanks.

Regards,

From: Jeff Morriss <jeff.morriss@xxxxxxxxxxx>
Subject: Re: [Ethereal-dev] packet-sccp.c patch for compilation under Win32
Date: Wed, 06 Mar 2002 12:07:44 -0500

> 
> Actually, that patch will break the intended functionality :(.
> 
> Does the attached patch fix VC++'s complaint?
> 
> If not, the offending statement should be broken into:
> 
> if (called)
> 	called_ssn = ssn;
> else
> 	calling_ssn = ssn;
> 
> Motonori Shindo wrote:
> > 
> > Hi,
> > 
> > The enclosed patch will stop Visual C++ to complain about the lvalue
> > of assignment.
> > 
> > Regards,
> > 
> > =--=--=--=--=--=--=--=--=--=--=--=--=--=--=--=--=
> >  +----+----+
> >  |.. .|    |     Motonori Shindo
> >  |_~__|    |
> >  | .. |~~_~|     Sr. Systems Engineer
> >  | .  |    |     CoSine Communications Inc.
> >  +----+----+
> >  C o S i n e     e-mail:  mshindo@xxxxxxxxxxxxx
> > Communications
> > =--=--=--=--=--=--=--=--=--=--=--=--=--=--=--=--=
> > 
> >   ------------------------------------------------------------------------
> > Index: packet-sccp.c
> > ===================================================================
> > RCS file: /cvsroot/ethereal/packet-sccp.c,v
> > retrieving revision 1.2
> > diff -u -r1.2 packet-sccp.c
> > --- packet-sccp.c       2002/03/05 00:09:21     1.2
> > +++ packet-sccp.c       2002/03/06 10:33:26
> > @@ -902,7 +902,7 @@
> >      /* Dissect SSN (if present) */
> >      if (ssni) {
> >        ssn = tvb_get_guint8(tvb, offset);
> > -      called ? called_ssn : calling_ssn = ssn;
> > +      called ? called_ssn : (calling_ssn = ssn);
> > 
> >        proto_tree_add_uint(call_tree, called ? hf_sccp_called_ssn
> >                                             : hf_sccp_calling_ssn,
> > @@ -944,7 +944,7 @@
> >      /* Dissect SSN (if present) */
> >      if (ssni) {
> >        ssn = tvb_get_guint8(tvb, offset);
> > -      called ? called_ssn : calling_ssn = ssn;
> > +      called ? called_ssn : (calling_ssn = ssn);
> > 
> >        proto_tree_add_uint(call_tree, called ? hf_sccp_called_ssn
> >                                             : hf_sccp_calling_ssn,
> > 
> >   ------------------------------------------------------------------------
> >                     Name: sanitizer.log
> >    sanitizer.log    Type: Text Document (application/x-unknown-content-type-txtfile)
> >                 Encoding: 8bit

=--=--=--=--=--=--=--=--=--=--=--=--=--=--=--=--=
 +----+----+     
 |.. .|    |     Motonori Shindo
 |_~__|    |     
 | .. |~~_~|     Sr. Systems Engineer
 | .  |    |     CoSine Communications Inc.
 +----+----+     
 C o S i n e     e-mail:  mshindo@xxxxxxxxxxxxx 
Communications
=--=--=--=--=--=--=--=--=--=--=--=--=--=--=--=--=