Navigation



Ignore:
Timestamp:
Apr 18, 2020, 2:16:18 PM (4 years ago)
Author:
Luke Mewburn <luke@mewburn.net>
Branch:
default
Phase:
public
Message:

dict_dcca_3gpp: highlight more local fixes

Mark local fixes with "Fix:"
Use "Table" vs "table".
Minor formatting consistency.

No change to generated dictionary.

File:
1 edited

Legend:

Unmodified
Added
Removed
  • extensions/dict_dcca_3gpp/add_avps.c

    r1528 r1530  
    5050        /*==================================================================*/
    5151
    52         /* Note: 3GPP-IMSI (1) changed to Must M.                           */
     52        /* Fix: 3GPP-IMSI (1) changed to Must M.                            */
    5353        /* 3GPP-IMSI, UTF8String, code 1, section 16.4.7                    */
    5454        {
     
    7979        };
    8080
    81         /* Note: 3GPP-PDP-Type (3) changed to Must M.                       */
     81        /* Fix: 3GPP-PDP-Type (3) changed to Must M.                        */
    8282        /* 3GPP-PDP-Type, Enumerated, code 3, section 16.4.7                */
    8383        {
     
    109109        };
    110110
    111         /* Note: 3GPP-GPRS-Negotiated-QoS-Profile (5) changed to Must M per contrib/3gpp. */
     111        /* Fix: 3GPP-GPRS-Negotiated-QoS-Profile (5) changed to Must M per contrib/3gpp. */
    112112        /* 3GPP-GPRS-Negotiated-QoS-Profile, UTF8String, code 5, section 16.4.7 */
    113113        {
     
    149149        };
    150150
    151         /* Note: 3GPP-IMSI-MCC-MNC (8) changed to Must M per packet dumps.  */
     151        /* Fix: 3GPP-IMSI-MCC-MNC (8) changed to Must M per packet dumps.   */
    152152        /* 3GPP-IMSI-MCC-MNC, UTF8String, code 8, section 16.4.7            */
    153153        {
     
    163163        };
    164164
    165         /* Note: 3GPP-GGSN-MCC-MNC (9) changed to Must M per contrib/3gpp.  */
     165        /* Fix: 3GPP-GGSN-MCC-MNC (9) changed to Must M per contrib/3gpp.   */
    166166        /* 3GPP-GGSN-MCC-MNC, UTF8String, code 9, section 16.4.7            */
    167167        {
     
    177177        };
    178178
    179         /* Note: 3GPP-NSAPI (10) changed to Must M per contrib/3gpp.        */
     179        /* Fix: 3GPP-NSAPI (10) changed to Must M per contrib/3gpp.         */
    180180        /* 3GPP-NSAPI, OctetString, code 10, section 16.4.7                 */
    181181        {
     
    191191        };
    192192
    193         /* Note: 3GPP TS 29.061 table 9a does not define                    */
     193        /* Note: 3GPP TS 29.061 Table 9a does not define                    */
    194194        /* 3GPP-Session-Stop-Indicator (11) even when it is referenced      */
    195195        /* by other 3GPP Diameter AVPs including PS-Information (874).      */
     196        /*                                                                  */
     197        /* Fix: Add 3GPP-Session-Stop-Indicator (11) OctetString.           */
    196198        /* 3GPP-Session-Stop-Indicator, OctetString, code 11, section 16.4.7 */
    197199        {
     
    207209        };
    208210
    209         /* Note: 3GPP-Selection-Mode (12) changed to Must M per contrib/3gpp. */
     211        /* Fix: 3GPP-Selection-Mode (12) changed to Must M per contrib/3gpp. */
    210212        /* 3GPP-Selection-Mode, UTF8String, code 12, section 16.4.7         */
    211213        {
     
    221223        };
    222224
    223         /* Note: 3GPP-Charging-Characteristics (13) changed to Must M per contrib/3gpp. */
     225        /* Fix: 3GPP-Charging-Characteristics (13) changed to Must M per contrib/3gpp. */
    224226        /* 3GPP-Charging-Characteristics, UTF8String, code 13, section 16.4.7 */
    225227        {
     
    238240        /* 3GPP-CG-IPv6-Address (14) to 3GPP-CG-Ipv6-Address;               */
    239241        /* this rename was not applied.                                     */
    240         /* 3GPP-CG-IPv6-Address (14) changed to Must M per contrib/3gpp.    */
     242        /*                                                                  */
     243        /* Fix: 3GPP-CG-IPv6-Address (14) changed to Must M per contrib/3gpp. */
    241244        /* 3GPP-CG-IPv6-Address, OctetString, code 14, section 16.4.7       */
    242245        {
     
    255258        /* 3GPP-SGSN-IPv6-Address (15) to 3GPP-SGSN-Ipv6-Address;           */
    256259        /* this rename was not applied.                                     */
    257         /* 3GPP-SGSN-IPv6-Address (15) changed to Must M per contrib/3gpp.  */
     260        /*                                                                  */
     261        /* Fix: 3GPP-SGSN-IPv6-Address (15) changed to Must M per contrib/3gpp. */
    258262        /* 3GPP-SGSN-IPv6-Address, OctetString, code 15, section 16.4.7     */
    259263        {
     
    272276        /* 3GPP-GGSN-IPv6-Address (16) to 3GPP-GGSN-Ipv6-Address;           */
    273277        /* this rename was not applied.                                     */
    274         /* 3GPP-GGSN-IPv6-Address (16) changed to Must M per contrib/3gpp.  */
     278        /*                                                                  */
     279        /* Fix: 3GPP-GGSN-IPv6-Address (16) changed to Must M per contrib/3gpp. */
    275280        /* 3GPP-GGSN-IPv6-Address, OctetString, code 16, section 16.4.7     */
    276281        {
     
    289294        /* 3GPP-IPv6-DNS-Servers (17) to 3GPP-Ipv6-DNS-Servers;             */
    290295        /* this rename was not applied.                                     */
    291         /* 3GPP-IPv6-DNS-Servers (17) changed to Must M per contrib/3gpp.   */
     296        /*                                                                  */
     297        /* Fix: 3GPP-IPv6-DNS-Servers (17) changed to Must M per contrib/3gpp. */
    292298        /* 3GPP-IPv6-DNS-Servers, OctetString, code 17, section 16.4.7      */
    293299        {
     
    303309        };
    304310
    305         /* Note: 3GPP-SGSN-MCC-MNC (18) changed to Must M per contrib/3gpp. */
     311        /* Fix: 3GPP-SGSN-MCC-MNC (18) changed to Must M per contrib/3gpp. */
    306312        /* 3GPP-SGSN-MCC-MNC, UTF8String, code 18, section 16.4.7           */
    307313        {
     
    317323        };
    318324
    319         /* Note: 3GPP-IMEISV (20) changed to Must M.                        */
     325        /* Fix: 3GPP-IMEISV (20) changed to Must M.                         */
    320326        /* 3GPP-IMEISV, OctetString, code 20, section 16.4.7                */
    321327        {
     
    331337        };
    332338
    333         /* Note: 3GPP-RAT-Type (21) changed to Must M.                      */
     339        /* Fix: 3GPP-RAT-Type (21) changed to Must M.                       */
    334340        /* 3GPP-RAT-Type, OctetString, code 21, section 16.4.7              */
    335341        {
     
    345351        };
    346352
    347         /* Note: 3GPP-User-Location-Info (22) changed to Must M.            */
     353        /* Fix: 3GPP-User-Location-Info (22) changed to Must M.             */
    348354        /* 3GPP-User-Location-Info, OctetString, code 22, section 16.4.7    */
    349355        {
     
    359365        };
    360366
    361         /* Note: 3GPP-MS-TimeZone (23) changed to Must M.                   */
     367        /* Fix: 3GPP-MS-TimeZone (23) changed to Must M.                    */
    362368        /* 3GPP-MS-TimeZone, OctetString, code 23, section 16.4.7           */
    363369        {
     
    425431        };
    426432
    427         /* Note: 3GPP TS 29.061 table 9a does not define                    */
     433        /* Note: 3GPP TS 29.061 Table 9a does not define                    */
    428434        /* External-Identifier (28) for Diameter; it is RADIUS only.        */
    429         /* 3GPP TS 29.061 table 9b refers to 3GPP TS 29.336                 */
     435        /* 3GPP TS 29.061 Table 9b refers to 3GPP TS 29.336                 */
    430436        /* for External-Identifier (3111).                                  */
    431437        /*                                                                  */
     
    925931        };
    926932
    927         /* Note: 3GPP TS 29.061 table 20.5a.1 row MBMS-Flag (931) has       */
     933        /* Note: 3GPP TS 29.061 Table 20.5a.1 row MBMS-Flag (931) has       */
    928934        /* incorrect name and should be "MBMS-Flags" per clause 20.5a.9.    */
    929935        /*                                                                  */
     
    10851091        };
    10861092
    1087         /* Note: 3GPP TS 29.109 table 6.1 row GBA_U-Awareness-Indicator (407) */
     1093        /* Note: 3GPP TS 29.109 Table 6.1 row GBA_U-Awareness-Indicator (407) */
    10881094        /* has an underscore in the name (contrary to RFC 6733 section 4.1). */
    10891095        /*                                                                  */
     
    40424048        };
    40434049
    4044         /* Note: 3GPP TS 29.212 table 5.3.0.1 row Max-PLR-DL (2852)         */
     4050        /* Note: 3GPP TS 29.212 Table 5.3.0.1 row Max-PLR-DL (2852)         */
    40454051        /* incorrectly lists type as Unsigned32 instead of Float32.         */
    40464052        /* 3GPP TS 29.212 clause 5.3.138 uses Float32.                      */
     
    40604066        };
    40614067
    4062         /* Note: 3GPP TS 29.212 table 5.3.0.1 row Max-PLR-UL (2853)         */
     4068        /* Note: 3GPP TS 29.212 Table 5.3.0.1 row Max-PLR-UL (2853)         */
    40634069        /* incorrectly lists type as Unsigned32 instead of Float32.         */
    40644070        /* 3GPP TS 29.212 clause 5.3.139 uses Float32.                      */
     
    43294335        };
    43304336
    4331         /* Note: 3GPP TS 29.212 table 5.3.0.1 row PDN-Connection-ID (1065)  */
    4332         /* missing M.                                                       */
     4337        /* Note: 3GPP TS 29.212 Table 5.3.0.1 row PDN-Connection-ID (1065)  */
     4338        /* missing M, assume MUST NOT.                                      */
     4339        /*                                                                  */
     4340        /* Fix: PDN-Connection-ID (1065) flag M MUST NOT.                   */
    43334341        /* PDN-Connection-ID, OctetString, code 1065, section 5.3.58        */
    43344342        {
     
    43374345                        10415,  /* Vendor */
    43384346                        "PDN-Connection-ID",    /* Name */
    4339                         AVP_FLAG_VENDOR,        /* Fixed flags */
     4347                        AVP_FLAG_VENDOR |AVP_FLAG_MANDATORY,    /* Fixed flags */
    43404348                        AVP_FLAG_VENDOR,        /* Fixed flag values */
    43414349                        AVP_TYPE_OCTETSTRING    /* base type of data */
     
    44934501        };
    44944502
    4495         /* Note: 3GPP TS 29.212 table 5.3.0.1 row PS-to-CS-Session-Continuity (1099) */
    4496         /* missing M.                                                       */
     4503        /* Note: 3GPP TS 29.212 Table 5.3.0.1 row PS-to-CS-Session-Continuity (1099) */
     4504        /* missing M, assume MUST NOT.                                      */
     4505        /*                                                                  */
     4506        /* Fix: PS-to-CS-Session-Continuity (1099) flag M MUST NOT.         */
    44974507        /* PS-to-CS-Session-Continuity, Enumerated, code 1099, section 5.3.84 */
    44984508        {
     
    45014511                        10415,  /* Vendor */
    45024512                        "PS-to-CS-Session-Continuity",  /* Name */
    4503                         AVP_FLAG_VENDOR,        /* Fixed flags */
     4513                        AVP_FLAG_VENDOR |AVP_FLAG_MANDATORY,    /* Fixed flags */
    45044514                        AVP_FLAG_VENDOR,        /* Fixed flag values */
    45054515                        AVP_TYPE_INTEGER32      /* base type of data */
     
    58525862        };
    58535863
    5854         /* Note: 3GPP TS 29.214 table 5.3.0.1 row IMS-Content-Identifier (563) */
     5864        /* Note: 3GPP TS 29.214 Table 5.3.0.1 row IMS-Content-Identifier (563) */
    58555865        /* missing M, assume MUST NOT.                                      */
    58565866        /*                                                                  */
     
    58695879        };
    58705880
    5871         /* Note: 3GPP TS 29.214 table 5.3.0.1 row IMS-Content (564)         */
     5881        /* Note: 3GPP TS 29.214 Table 5.3.0.1 row IMS-Content (564)         */
    58725882        /* missing M, assume MUST NOT.                                      */
    58735883        /*                                                                  */
     
    89148924        };
    89158925
    8916         /* Note: 3GPP TS 29.272 V12.5.0 (2014-06) table 7.3.1/1             */
     8926        /* Note: 3GPP TS 29.272 V12.5.0 (2014-06) Table 7.3.1/1             */
    89178927        /* changed row SS-Status (1477) to be OctetString instead of Grouped. */
    89188928        /* Clause 7.3.88 already described SS-Status (1147) as OctetString. */
     
    92459255        };
    92469256
    9247         /* Note: 3GPP TS 29.272 table 7.3.1/1 row MME-Location-Information (1600) */
     9257        /* Note: 3GPP TS 29.272 Table 7.3.1/1 row MME-Location-Information (1600) */
    92489258        /* incorrectly has a space instead of hyphen in the name.           */
    92499259        /*                                                                  */
     
    1032410334        };
    1032510335
    10326         /* Note: 3GPP TS 29.272 table 7.3.1/1 row Additional-Context-ID (1683) has */
     10336        /* Note: 3GPP TS 29.272 Table 7.3.1/1 row Additional-Context-ID (1683) has */
    1032710337        /* incorrect name and should be "Additional-Context-Identifier" per clause 7.3.206. */
    1032810338        /*                                                                  */
     
    1036710377        };
    1036810378
    10369         /* Note: 3GPP TS 29.272 table 7.3.1/1 row Preferred-Data-Mode (1686) */
     10379        /* Note: 3GPP TS 29.272 Table 7.3.1/1 row Preferred-Data-Mode (1686) */
    1037010380        /* incorrectly has value type Grouped instead of Unsigned32,        */
    1037110381        /* conflicting with clause 7.3.209.                                 */
     10382        /*                                                                  */
     10383        /* Fix: Preferred-Data-Mode (1686) type changed from Grouped to Unsigned32. */
    1037210384        /* Preferred-Data-Mode, Unsigned32, code 1686, section 7.3.209      */
    1037310385        {
     
    1333613348        };
    1333713349
    13338         /* Note: 3GPP TS 29.338 table 6.3.3.1/1 row HSS-ID (3325)           */
     13350        /* Note: 3GPP TS 29.338 Table 6.3.3.1/1 row HSS-ID (3325)           */
    1333913351        /* incorrectly lists type as OctetString instead of UTF8String.     */
    1334013352        /* 3GPP TS 29.338 clause 6.3.3.14 uses UTF8String.                  */
     
    1371813730        };
    1371913731
    13720         /* Note: 3GPP TS 29.344 table 6.3.1-1 missing Authorized-Discovery-Range. */
     13732        /* Note: 3GPP TS 29.344 Table 6.3.1-1 missing Authorized-Discovery-Range. */
    1372113733        /* Present in 3GPP TS 29.344 clause 6.3.13 and 3GPP TS 29.230 Table 7.1. */
    13722 
     13734        /*                                                                  */
    1372313735        /* Fix: Add Authorized-Discovery-Range (3708) Unsigned32.           */
    1372413736        /* Authorized-Discovery-Range, Unsigned32, code 3708, section 6.3.13 */
     
    1783417846        };
    1783517847
    17836         /* Note: 3GPP TS 32.299 V11.8.0 (2013-07) table 7.2 changed         */
     17848        /* Note: 3GPP TS 32.299 V11.8.0 (2013-07) Table 7.2 changed         */
    1783717849        /* row MBMS-GW-Address (2307) to have a hyphen instead of space.    */
    1783817850        /*                                                                  */
Note: See TracChangeset for help on using the changeset viewer.