those warnings are really pedantic. But anyway, here is the patch.
Regards,
Lars
Joerg Mayer wrote:
On Fri, Feb 06, 2004 at 09:11:39AM +0100, Ulf Lamping wrote:
Some new(?) MSVC compiler warnings:
packet-h225.c(10326) : warning C4018: '>' : Konflikt zwischen signed und unsigned packet-h225.c(10328) : warning C4018: '>' : Konflikt zwischen signed und unsigned
Here's a more complete list (from gcc with extra warnings):
packet-h225.c:1282: warning: pointer targets in passing arg 9 of `dissect_per_choice' differ in signedness packet-h225.c:1571: warning: pointer targets in passing arg 9 of `dissect_per_choice' differ in signedness packet-h225.c:1604: warning: pointer targets in passing arg 9 of `dissect_per_choice' differ in signedness packet-h225.c:1637: warning: pointer targets in passing arg 9 of `dissect_per_choice' differ in signedness packet-h225.c:1746: warning: pointer targets in passing arg 9 of `dissect_per_choice' differ in signedness packet-h225.c:1770: warning: pointer targets in passing arg 9 of `dissect_per_choice' differ in signedness packet-h225.c:1797: warning: pointer targets in passing arg 9 of `dissect_per_choice' differ in signedness packet-h225.c:1825: warning: pointer targets in passing arg 9 of `dissect_per_choice' differ in signedness packet-h225.c:2395: warning: pointer targets in passing arg 9 of `dissect_per_choice' differ in signedness packet-h225.c:5152: warning: pointer targets in passing arg 9 of `dissect_per_choice' differ in signedness packet-h225.c:5310: warning: pointer targets in passing arg 9 of `dissect_per_choice' differ in signedness packet-h225.c:6099: warning: pointer targets in passing arg 9 of `dissect_per_choice' differ in signedness packet-h225.c:10326: warning: comparison between signed and unsigned packet-h225.c:10328: warning: comparison between signed and unsigned
Index: packet-h225.c
===================================================================
RCS file: /cvsroot/ethereal/packet-h225.c,v
retrieving revision 1.28
diff -u -r1.28 packet-h225.c
--- packet-h225.c 5 Feb 2004 19:19:05 -0000 1.28
+++ packet-h225.c 6 Feb 2004 17:38:20 -0000
@@ -1279,7 +1279,10 @@
static int
dissect_h225_FacilityReason(tvbuff_t *tvb, int offset, packet_info *pinfo, proto_tree
*tree)
{
- offset=dissect_per_choice(tvb, offset, pinfo, tree, hf_h225_FacilityReason,
ett_h225_FacilityReason, FacilityReason_choice, "FacilityReason", &(h225_pi.reason));
+ guint32 value;
+
+ offset=dissect_per_choice(tvb, offset, pinfo, tree, hf_h225_FacilityReason,
ett_h225_FacilityReason, FacilityReason_choice, "FacilityReason", &value);
+ h225_pi.reason = value;
return offset;
}
@@ -1568,7 +1571,10 @@
static int
dissect_h225_GatekeeperRejectReason(tvbuff_t *tvb, int offset, packet_info *pinfo,
proto_tree *tree)
{
- offset=dissect_per_choice(tvb, offset, pinfo, tree,
hf_h225_GatekeeperRejectReason, ett_h225_GatekeeperRejectReason,
GatekeeperRejectReason_choice, "GatekeeperRejectReason", &(h225_pi.reason));
+ guint32 value;
+
+ offset=dissect_per_choice(tvb, offset, pinfo, tree,
hf_h225_GatekeeperRejectReason, ett_h225_GatekeeperRejectReason,
GatekeeperRejectReason_choice, "GatekeeperRejectReason", &value);
+ h225_pi.reason = value;
return offset;
}
@@ -1601,7 +1607,10 @@
static int
dissect_h225_UnregRequestReason(tvbuff_t *tvb, int offset, packet_info *pinfo,
proto_tree *tree)
{
- offset=dissect_per_choice(tvb, offset, pinfo, tree,
hf_h225_UnregRequestReason, ett_h225_UnregRequestReason, UnregRequestReason_choice,
"UnregRequestReason", &(h225_pi.reason));
+ guint32 value;
+
+ offset=dissect_per_choice(tvb, offset, pinfo, tree,
hf_h225_UnregRequestReason, ett_h225_UnregRequestReason, UnregRequestReason_choice,
"UnregRequestReason", &value);
+ h225_pi.reason = value;
return offset;
}
@@ -1634,7 +1643,10 @@
static int
dissect_h225_UnregRejectReason(tvbuff_t *tvb, int offset, packet_info *pinfo,
proto_tree *tree)
{
- offset=dissect_per_choice(tvb, offset, pinfo, tree, hf_h225_UnregRejectReason,
ett_h225_UnregRejectReason, UnregRejectReason_choice, "UnregRejectReason",
&(h225_pi.reason));
+ guint32 value;
+
+ offset=dissect_per_choice(tvb, offset, pinfo, tree, hf_h225_UnregRejectReason,
ett_h225_UnregRejectReason, UnregRejectReason_choice, "UnregRejectReason", &value);
+ h225_pi.reason = value;
return offset;
}
@@ -1743,7 +1755,10 @@
static int
dissect_h225_BandRejectReason(tvbuff_t *tvb, int offset, packet_info *pinfo,
proto_tree *tree)
{
- offset=dissect_per_choice(tvb, offset, pinfo, tree, hf_h225_BandRejectReason,
ett_h225_BandRejectReason, BandRejectReason_choice, "BandRejectReason",
&(h225_pi.reason));
+ guint32 value;
+
+ offset=dissect_per_choice(tvb, offset, pinfo, tree, hf_h225_BandRejectReason,
ett_h225_BandRejectReason, BandRejectReason_choice, "BandRejectReason", &value);
+ h225_pi.reason = value;
return offset;
}
@@ -1767,7 +1782,10 @@
static int
dissect_h225_DisengageReason(tvbuff_t *tvb, int offset, packet_info *pinfo,
proto_tree *tree)
{
- offset=dissect_per_choice(tvb, offset, pinfo, tree, hf_h225_DisengageReason,
ett_h225_DisengageReason, DisengageReason_choice, "DisengageReason",
&(h225_pi.reason));
+ guint32 value;
+
+ offset=dissect_per_choice(tvb, offset, pinfo, tree, hf_h225_DisengageReason,
ett_h225_DisengageReason, DisengageReason_choice, "DisengageReason", &value);
+ h225_pi.reason = value;
return offset;
}
@@ -1794,7 +1812,10 @@
static int
dissect_h225_DisengageRejectReason(tvbuff_t *tvb, int offset, packet_info *pinfo,
proto_tree *tree)
{
- offset=dissect_per_choice(tvb, offset, pinfo, tree,
hf_h225_DisengageRejectReason, ett_h225_DisengageRejectReason,
DisengageRejectReason_choice, "DisengageRejectReason", &(h225_pi.reason));
+ guint32 value;
+
+ offset=dissect_per_choice(tvb, offset, pinfo, tree,
hf_h225_DisengageRejectReason, ett_h225_DisengageRejectReason,
DisengageRejectReason_choice, "DisengageRejectReason", &value);
+ h225_pi.reason = value;
return offset;
}
@@ -1822,7 +1843,10 @@
static int
dissect_h225_InfoRequestNakReason(tvbuff_t *tvb, int offset, packet_info *pinfo,
proto_tree *tree)
{
- offset=dissect_per_choice(tvb, offset, pinfo, tree,
hf_h225_InfoRequestNakReason, ett_h225_InfoRequestNakReason,
InfoRequestNakReason_choice, "InfoRequestNakReason", &(h225_pi.reason));
+ guint32 value;
+
+ offset=dissect_per_choice(tvb, offset, pinfo, tree,
hf_h225_InfoRequestNakReason, ett_h225_InfoRequestNakReason,
InfoRequestNakReason_choice, "InfoRequestNakReason", &value);
+ h225_pi.reason = value;
return offset;
}
@@ -2392,7 +2416,10 @@
static int
dissect_h225_ReleaseCompleteReason(tvbuff_t *tvb, int offset, packet_info *pinfo,
proto_tree *tree)
{
- offset=dissect_per_choice(tvb, offset, pinfo, tree,
hf_h225_ReleaseCompleteReason, ett_h225_ReleaseCompleteReason,
ReleaseCompleteReason_choice, "ReleaseCompleteReason", &(h225_pi.reason));
+ guint32 value;
+
+ offset=dissect_per_choice(tvb, offset, pinfo, tree,
hf_h225_ReleaseCompleteReason, ett_h225_ReleaseCompleteReason,
ReleaseCompleteReason_choice, "ReleaseCompleteReason", &value);
+ h225_pi.reason = value;
return offset;
}
@@ -5149,7 +5176,10 @@
static int
dissect_h225_AdmissionRejectReason(tvbuff_t *tvb, int offset, packet_info *pinfo,
proto_tree *tree)
{
- offset=dissect_per_choice(tvb, offset, pinfo, tree,
hf_h225_AdmissionRejectReason, ett_h225_AdmissionRejectReason,
AdmissionRejectReason_choice, "AdmissionRejectReason", &(h225_pi.reason));
+ guint32 value;
+
+ offset=dissect_per_choice(tvb, offset, pinfo, tree,
hf_h225_AdmissionRejectReason, ett_h225_AdmissionRejectReason,
AdmissionRejectReason_choice, "AdmissionRejectReason", &value);
+ h225_pi.reason = value;
return offset;
}
@@ -5307,7 +5337,10 @@
static int
dissect_h225_LocationRejectReason(tvbuff_t *tvb, int offset, packet_info *pinfo,
proto_tree *tree)
{
- offset=dissect_per_choice(tvb, offset, pinfo, tree,
hf_h225_LocationRejectReason, ett_h225_LocationRejectReason,
LocationRejectReason_choice, "LocationRejectReason", &(h225_pi.reason));
+ guint32 value;
+
+ offset=dissect_per_choice(tvb, offset, pinfo, tree,
hf_h225_LocationRejectReason, ett_h225_LocationRejectReason,
LocationRejectReason_choice, "LocationRejectReason", &value);
+ h225_pi.reason = value;
return offset;
}
@@ -6096,7 +6129,10 @@
static int
dissect_h225_RegistrationRejectReason(tvbuff_t *tvb, int offset, packet_info *pinfo,
proto_tree *tree)
{
- offset=dissect_per_choice(tvb, offset, pinfo, tree,
hf_h225_RegistrationRejectReason, ett_h225_RegistrationRejectReason,
RegistrationRejectReason_choice, "RegistrationRejectReason", &(h225_pi.reason));
+ guint32 value;
+
+ offset=dissect_per_choice(tvb, offset, pinfo, tree,
hf_h225_RegistrationRejectReason, ett_h225_RegistrationRejectReason,
RegistrationRejectReason_choice, "RegistrationRejectReason", &value);
+ h225_pi.reason = value;
return offset;
}
@@ -10323,9 +10359,9 @@
/* if end of list is reached, exit loop and
decide if request is duplicate or not. */
if (h225ras_call->next_call == NULL) {
if ( (pinfo->fd->num >
h225ras_call->rsp_num && h225ras_call->rsp_num != 0
- && pinfo->fd->abs_secs >
h225ras_call->req_time.secs + THRESHOLD_REPEATED_RESPONDED_CALL )
+ && pinfo->fd->abs_secs > (guint)
(h225ras_call->req_time.secs + THRESHOLD_REPEATED_RESPONDED_CALL) )
||(pinfo->fd->num >
h225ras_call->req_num && h225ras_call->rsp_num == 0
- && pinfo->fd->abs_secs >
h225ras_call->req_time.secs + THRESHOLD_REPEATED_NOT_RESPONDED_CALL ) )
+ && pinfo->fd->abs_secs > (guint)
(h225ras_call->req_time.secs + THRESHOLD_REPEATED_NOT_RESPONDED_CALL) ) )
{
/* if last request has been
responded
and this request appears
after last response (has bigger frame number)
_______________________________________________ Ethereal-dev mailing list [EMAIL PROTECTED] http://www.ethereal.com/mailman/listinfo/ethereal-dev
