K2InformaticsGmbH / smpp_parser

A parser for SMPP protocol PDUs
Apache License 2.0
3 stars 1 forks source link

Invalid TLV Value #47

Closed walter-weinmann closed 6 years ago

walter-weinmann commented 6 years ago
=== Reason: no match of right hand side value {error,
                                                    <<"Invalid TLV Value">>}
  in function  smpp_test_utils:encode_decode/1 (d:/K2/smpp_parser_idea/smpp_parser/_build/test/lib/smpp_parser/test/src/smpp_test_utils.erl, line 47)
  in call from compacted_deliver_sm_SUITE:test_compacted/1 (d:/K2/smpp_parser_idea/smpp_parser/_build/test/lib/smpp_parser/test/generated/ct/compacted_deliver_sm_SUITE.erl, line 49)
  in call from test_server:ts_tc/3 (test_server.erl, line 1546)
  in call from test_server:run_test_case_eval1/6 (test_server.erl, line 1062)
  in call from test_server:run_test_case_eval/9 (test_server.erl, line 994)

Example:

 {"deliver_sm_issue_47",
  "00 00 01 60 00 00 00 05 00 00 00 00 00 00 00 01 57 41 50 00 01 06 31 36 38 "
  "2E 31 32 33 2E 32 33 34 2E 33 32 31 00 01 06 31 32 37 2E 30 2E 30 2E 31 00 "
  "01 03 00 39 39 30 37 32 34 31 37 35 34 34 34 30 30 30 52 00 39 39 30 36 32 "
  "35 31 38 35 35 34 35 30 30 30 2B 00 10 01 02 FF FF 31 20 54 68 69 73 20 69 "
  "73 20 61 20 73 68 6F 72 74 20 6D 65 73 73 61 67 65 32 20 54 68 69 73 20 69 "
  "73 20 61 20 73 68 6F 72 74 20 6D 65 73 73 61 67 65 33 20 54 68 69 73 20 69 "
  "73 20 61 20 73 68 6F 72 74 20 6D 65 73 73 61 67 65 34 20 54 68 69 73 20 69 "
  "73 20 61 20 73 68 6F 72 74 20 6D 65 73 73 61 67 65 35 20 54 68 69 73 20 69 "
  "73 20 61 20 73 68 6F 72 74 20 6D 65 73 73 61 67 65 36 20 54 68 69 73 20 69 "
  "73 20 61 20 73 68 6F 72 74 20 6D 65 73 73 61 67 65 37 20 54 68 69 73 20 69 "
  "73 20 61 20 73 68 6F 72 74 20 6D 65 73 73 61 67 65 38 20 54 68 69 73 20 69 "
  "73 20 61 20 73 68 6F 72 74 20 6D 65 73 73 61 67 65 39 20 54 68 69 73 20 69 "
  "73 20 61 20 73 68 6F 72 74 20 6D 65 73 73 61 67 65 41 20 54 68 69 73 20 69 "
  "73 20 61 20 73 68 6F 72 74 20 6D 65 73 73 61 67 65 42 20 45 6E 64 02 05 00 "
  "01 FE"}
c-bik commented 6 years ago

Seems like it got fixed too (as side effert of other fixes perhaps)!

@walter-weinmann can you please verify / confirm?