diff options
| author | Jeff Morriss <jeff.morriss@ulticom.com> | 2008-03-15 22:41:57 +0000 |
|---|---|---|
| committer | Jeff Morriss <jeff.morriss@ulticom.com> | 2008-03-15 22:41:57 +0000 |
| commit | 689e8bd8559fec00503b4563af219c2c842101e3 (patch) | |
| tree | f47ad51a0b89e8928364c8ae55f70a5d16d8dfdf /epan/dissectors/packet-pcnfsd.c | |
| parent | 320c1117f60b6d777043d9e2ac01dcbcc1ca95cd (diff) | |
From http://bugs.wireshark.org/bugzilla/show_bug.cgi?id=1539 :
Hexadecimal and octal are unsigned. Don't let dissectors register signed
fields (FT_INT*) to be displayed in hexadecimal (including HEX_DEC and DEC_HEX)
or octal. Fix dissectors that do that mostly by changing the fields to
unsigned though in PANA it appears the fields are meant to be signed so
change those fields to be displayed in decimal.
This fixes an assertion crash in hfinfo_numeric_format() if/when someone tries
to create a filter using one of these mixed signed/unsigned fields (because
that routine does not know how to present the user with a signed value in
hex).
Also add FT_*INT64 to the "make sure it's not BASE_NONE" check.
svn path=/trunk/; revision=24643
Diffstat (limited to 'epan/dissectors/packet-pcnfsd.c')
| -rw-r--r-- | epan/dissectors/packet-pcnfsd.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/epan/dissectors/packet-pcnfsd.c b/epan/dissectors/packet-pcnfsd.c index 5dc24022a9..d1962ec9b6 100644 --- a/epan/dissectors/packet-pcnfsd.c +++ b/epan/dissectors/packet-pcnfsd.c @@ -402,7 +402,7 @@ proto_register_pcnfsd(void) "Home Directory", "pcnfsd.homedir", FT_STRING, BASE_DEC, NULL, 0, "Home Directory", HFILL }}, { &hf_pcnfsd_def_umask, { - "def_umask", "pcnfsd.def_umask", FT_INT32, BASE_OCT, + "def_umask", "pcnfsd.def_umask", FT_UINT32, BASE_OCT, NULL, 0, "def_umask", HFILL }}, { &hf_pcnfsd_username, { "User name", "pcnfsd.username", FT_STRING, BASE_DEC, |