Menu ▾ ▴

#109 saAmfComponentErrorReport_4 succeds only when the local component on the node is given

5.20.08
fixed
nobody
None
enhancement
amf
nd
4.4
minor
False
2020-08-09
2013-05-13
Nagendra Kumar
No

Migrated from http://devel.opensaf.org/ticket/2354

When saAmfComponentErrorReport_4 is called with component on different node, api returns with SA_AIS_ERR_NOT_EXIST, instead of expected value SA_AIS_OK

There will be usecase where a component on one node needs to report failure of component on another node.

e.g. snmp agent component needs to report failure of snmpd, which is modelled as a component on another node.

Changed 16 months ago by nagendra ¶
■type changed from defect to enhancement
Documented in
changeset: 35:b4fd0969aa71
tag: tip
user: Nagendrankumar@…
date: Tue Jan 17 14:51:11 2012 +0530
summary: AVSV: saAmfComponentErrorReport_4 only supported for local node erroneous component.(#2354)

Related

Wiki: ChangeLog-5.20.08

Discussion

  • Nagendra Kumar

    Nagendra Kumar - 2013-05-13
    • Priority: major --> minor
     

  • Anand Sundararaj

    Anand Sundararaj - 2020-07-21
    • status: unassigned --> accepted
    • assigned_to: Anand Sundararaj
    • Blocker: --> False
    • Milestone: future --> 5.20.08
     

  • Anand Sundararaj

    Anand Sundararaj - 2020-07-24
    • status: accepted --> review
    • Part: - --> nd
     

  • Minh Hon Chau

    Minh Hon Chau - 2020-08-05

    2020-08-05 02:23:26,079 INFO - *** cpplint errors removed by patch: 48
    2020-08-05 02:23:26,079 INFO - *** cpplint errors added by patch: 77
    2020-08-05 02:23:26,079 INFO - *** List of cpplint errors removed by patch ***
    2020-08-05 02:23:26,079 INFO - *** src/amf/amfnd/amfnd.cc:278: Using C-style cast. Use reinterpret_cast<uint8_t *="">(...) instead [readability/casting] [4]
    2020-08-05 02:23:26,080 INFO - *** src/amf/amfnd/amfnd.cc:387: Lines should be <= 80 characters long [whitespace/line_length] [2]
    2020-08-05 02:23:26,080 INFO - *** src/amf/amfnd/avnd_cb.h:151: Is this a non-const reference? If so, make const or use a pointer: AVND_CB &cb [runtime/references] [2]
    2020-08-05 02:23:26,080 INFO - *** src/amf/amfnd/avnd_cb.h:155: Add #include <string> for string [build/include_what_you_use] [4]
    2020-08-05 02:23:26,080 INFO - *** src/amf/amfnd/avnd_comp.h:1010: Is this a non-const reference? If so, make const or use a pointer: AmfDb<std::string, avnd_comp=""> &compdb [runtime/references] [2]
    2020-08-05 02:23:26,080 INFO - *** src/amf/amfnd/avnd_comp.h:1013: Is this a non-const reference? If so, make const or use a pointer: AmfDb<std::string, avnd_comp=""> &compdb [runtime/references] [2]
    2020-08-05 02:23:26,080 INFO - *** src/amf/amfnd/avnd_comp.h:1037: Add #include <string> for string [build/include_what_you_use] [4]
    2020-08-05 02:23:26,081 INFO - *** src/amf/amfnd/avnd_comp.h:381: Complex multi-line /.../-style comment found. Lint may give bogus warnings. Consider replacing these with //-style comments, with #if 0...#endif, or with more clearly structured multi-line comments. [readability/multiline_comment] [5]
    2020-08-05 02:23:26,081 INFO - *** src/amf/amfnd/avnd_comp.h:403: Complex multi-line /.../-style comment found. Lint may give bogus warnings. Consider replacing these with //-style comments, with #if 0...#endif, or with more clearly structured multi-line comments. [readability/multiline_comment] [5]
    2020-08-05 02:23:26,081 INFO - *** src/amf/amfnd/avnd_comp.h:407: Complex multi-line /.../-style comment found. Lint may give bogus warnings. Consider replacing these with //-style comments, with #if 0...#endif, or with more clearly structured multi-line comments. [readability/multiline_comment] [5]
    2020-08-05 02:23:26,081 INFO - *** src/amf/amfnd/avnd_comp.h:407: Lines should be <= 80 characters long [whitespace/line_length] [2]
    2020-08-05 02:23:26,081 INFO - *** src/amf/amfnd/avnd_comp.h:414: Use operator || instead of or [readability/alt_tokens] [2]
    2020-08-05 02:23:26,081 INFO - *** src/amf/amfnd/avnd_comp.h:478: Complex multi-line /.../-style comment found. Lint may give bogus warnings. Consider replacing these with //-style comments, with #if 0...#endif, or with more clearly structured multi-line comments. [readability/multiline_comment] [5]
    2020-08-05 02:23:26,082 INFO - *** src/amf/amfnd/avnd_comp.h:479: Use operator || instead of or [readability/alt_tokens] [2]
    2020-08-05 02:23:26,082 INFO - *** src/amf/amfnd/avnd_comp.h:506: Lines should be <= 80 characters long [whitespace/line_length] [2]
    2020-08-05 02:23:26,082 INFO - *** src/amf/amfnd/avnd_comp.h:507: Lines should be <= 80 characters long [whitespace/line_length] [2]
    2020-08-05 02:23:26,082 INFO - *** src/amf/amfnd/avnd_comp.h:628: Using C-style cast. Use reinterpret_cast<uint8_t *="">(...) instead [readability/casting] [4]
    2020-08-05 02:23:26,082 INFO - *** src/amf/amfnd/avnd_comp.h:630: Using C-style cast. Use reinterpret_cast<avnd_comp *="">(...) instead [readability/casting] [4]
    2020-08-05 02:23:26,083 INFO - *** src/amf/amfnd/avnd_comp.h:634: Using C-style cast. Use reinterpret_cast<uint8_t *="">(...) instead [readability/casting] [4]
    2020-08-05 02:23:26,083 INFO - *** src/amf/amfnd/avnd_comp.h:635: Using C-style cast. Use reinterpret_cast<uint8_t *="">(...) instead [readability/casting] [4]
    2020-08-05 02:23:26,083 INFO - *** src/amf/amfnd/avnd_comp.h:635: Weird number of spaces at line-start. Are you using a 2-space indent? [whitespace/indent] [3]
    2020-08-05 02:23:26,083 INFO - *** src/amf/amfnd/avnd_comp.h:637: Using C-style cast. Use reinterpret_cast<avnd_comp_csi_rec *="">(...) instead [readability/casting] [4]
    2020-08-05 02:23:26,083 INFO - *** src/amf/amfnd/avnd_comp.h:644: Using C-style cast. Use reinterpret_cast<uint8_t *="">(...) instead [readability/casting] [4]
    2020-08-05 02:23:26,083 INFO - *** src/amf/amfnd/avnd_comp.h:655: Using C-style cast. Use reinterpret_cast<uint8_t *="">(...) instead [readability/casting] [4]
    2020-08-05 02:23:26,083 INFO - *** src/amf/amfnd/avnd_comp.h:665: Using C-style cast. Use reinterpret_cast<uint8_t *="">(...) instead [readability/casting] [4]
    2020-08-05 02:23:26,084 INFO - *** src/amf/amfnd/avnd_comp.h:675: Using C-style cast. Use reinterpret_cast<avnd_comp_hc_rec *="">(...) instead [readability/casting] [4]
    2020-08-05 02:23:26,084 INFO - *** src/amf/amfnd/avnd_comp.h:676: Using C-style cast. Use reinterpret_cast<uint8_t *="">(...) instead [readability/casting] [4]
    2020-08-05 02:23:26,084 INFO - *** src/amf/amfnd/avnd_comp.h:690: Is this a non-const reference? If so, make const or use a pointer: uint32_t &o_rc [runtime/references] [2]
    2020-08-05 02:23:26,084 INFO - *** src/amf/amfnd/avnd_comp.h:697: Almost always, snprintf is better than strcpy [runtime/printf] [4]
    2020-08-05 02:23:26,084 INFO - *** src/amf/amfnd/avnd_comp.h:699: Consider using strtok_r(...) instead of strtok(...) for improved thread safety. [runtime/threadsafe_fn] [2]
    2020-08-05 02:23:26,084 INFO - *** src/amf/amfnd/avnd_comp.h:704: Consider using strtok_r(...) instead of strtok(...) for improved thread safety. [runtime/threadsafe_fn] [2]
    2020-08-05 02:23:26,084 INFO - *** src/amf/amfnd/avnd_comp.h:707: Almost always, snprintf is better than strcpy [runtime/printf] [4]
    2020-08-05 02:23:26,085 INFO - *** src/amf/amfnd/avnd_comp.h:712: Use int16/int64/etc, rather than the C type long [runtime/int] [4]
    2020-08-05 02:23:26,085 INFO - *** src/amf/amfnd/avnd_comp.h:712: Using C-style cast. Use static_cast<char>(...) instead [readability/casting] [4]
    2020-08-05 02:23:26,085 INFO - *** src/amf/amfnd/avnd_comp.h:715: If an else has a brace on one side, it should have it on both [readability/braces] [5]
    2020-08-05 02:23:26,085 INFO - *** src/amf/amfnd/avnd_comp.h:730: Using C-style cast. Use reinterpret_cast<uint32_t *="">(...) instead [readability/casting] [4]
    2020-08-05 02:23:26,085 INFO - *** src/amf/amfnd/avnd_comp.h:797: If an else has a brace on one side, it should have it on both [readability/braces] [5]
    2020-08-05 02:23:26,085 INFO - *** src/amf/amfnd/avnd_comp.h:812: If an else has a brace on one side, it should have it on both [readability/braces] [5]
    2020-08-05 02:23:26,085 INFO - *** src/amf/amfnd/avnd_comp.h:819: Is this a non-const reference? If so, make const or use a pointer: uint32_t& o_found [runtime/references] [2]
    2020-08-05 02:23:26,086 INFO - *** src/amf/amfnd/avnd_comp.h:824: Line contains only semicolon. If this should be an empty statement, use {} instead. [whitespace/semicolon] [5]
    2020-08-05 02:23:26,086 INFO - *** src/amf/amfnd/clm.cc:258: If an else has a brace on one side, it should have it on both [readability/braces] [5]
    2020-08-05 02:23:26,086 INFO - *** src/amf/amfnd/err.cc:1207: Redundant blank line at the end of a code block should be deleted. [whitespace/blank_line] [3]
    2020-08-05 02:23:26,086 INFO - *** src/amf/amfnd/err.cc:1313: If an else has a brace on one side, it should have it on both [readability/braces] [5]
    2020-08-05 02:23:26,086 INFO - *** src/amf/amfnd/err.cc:391: Lines should be <= 80 characters long [whitespace/line_length] [2]
    2020-08-05 02:23:26,086 INFO - *** src/amf/amfnd/err.cc:456: If an else has a brace on one side, it should have it on both [readability/braces] [5]
    2020-08-05 02:23:26,087 INFO - *** src/amf/amfnd/err.cc:730: If an else has a brace on one side, it should have it on both [readability/braces] [5]
    2020-08-05 02:23:26,087 INFO - *** src/amf/amfnd/err.cc:890: Missing username in TODO; it should look like "// TODO(my_username): Stuff." [readability/todo] [2]
    2020-08-05 02:23:26,087 INFO - *** src/amf/amfnd/imm.cc:76: Using C-style cast. Use reinterpret_cast<avnd_evt *="">(...) instead [readability/casting] [4]</avnd_evt></uint32_t></char></uint8_t></avnd_comp_hc_rec></uint8_t></uint8_t></uint8_t></avnd_comp_csi_rec></uint8_t></uint8_t></avnd_comp></uint8_t></string></std::string,></std::string,></string></uint8_t>

    2020-08-05 02:23:26,087 INFO - *** List of cpplint errors added by patch ***
    2020-08-05 02:23:26,087 INFO - *** src/amf/amfnd/amfnd.cc:212: If an else has a brace on one side, it should have it on both [readability/braces] [5]
    2020-08-05 02:23:26,087 INFO - *** src/amf/amfnd/amfnd.cc:282: Lines should be <= 80 characters long [whitespace/line_length] [2]
    2020-08-05 02:23:26,087 INFO - *** src/amf/amfnd/amfnd.cc:282: Using C-style cast. Use reinterpret_cast<uint8_t *="">(...) instead [readability/casting] [4]
    2020-08-05 02:23:26,087 INFO - *** src/amf/amfnd/amfnd.cc:395: Lines should be <= 80 characters long [whitespace/line_length] [2]
    2020-08-05 02:23:26,087 INFO - *** src/amf/amfnd/avnd_cb.h:153: Is this a non-const reference? If so, make const or use a pointer: AVND_CB &cb [runtime/references] [2]
    2020-08-05 02:23:26,087 INFO - *** src/amf/amfnd/avnd_cb.h:157: Add #include <string> for string [build/include_what_you_use] [4]
    2020-08-05 02:23:26,087 INFO - *** src/amf/amfnd/avnd_comp.h:1012: Is this a non-const reference? If so, make const or use a pointer: AmfDb<std::string, avnd_comp=""> &compdb [runtime/references] [2]
    2020-08-05 02:23:26,087 INFO - *** src/amf/amfnd/avnd_comp.h:1015: Is this a non-const reference? If so, make const or use a pointer: AmfDb<std::string, avnd_comp=""> &compdb [runtime/references] [2]
    2020-08-05 02:23:26,088 INFO - *** src/amf/amfnd/avnd_comp.h:1039: Add #include <string> for string [build/include_what_you_use] [4]
    2020-08-05 02:23:26,088 INFO - *** src/amf/amfnd/avnd_comp.h:383: Complex multi-line /.../-style comment found. Lint may give bogus warnings. Consider replacing these with //-style comments, with #if 0...#endif, or with more clearly structured multi-line comments. [readability/multiline_comment] [5]
    2020-08-05 02:23:26,088 INFO - *** src/amf/amfnd/avnd_comp.h:405: Complex multi-line /.../-style comment found. Lint may give bogus warnings. Consider replacing these with //-style comments, with #if 0...#endif, or with more clearly structured multi-line comments. [readability/multiline_comment] [5]
    2020-08-05 02:23:26,088 INFO - *** src/amf/amfnd/avnd_comp.h:411: Lines should be <= 80 characters long [whitespace/line_length] [2]
    2020-08-05 02:23:26,088 INFO - *** src/amf/amfnd/avnd_comp.h:415: Complex multi-line /.../-style comment found. Lint may give bogus warnings. Consider replacing these with //-style comments, with #if 0...#endif, or with more clearly structured multi-line comments. [readability/multiline_comment] [5]
    2020-08-05 02:23:26,088 INFO - *** src/amf/amfnd/avnd_comp.h:416: Use operator || instead of or [readability/alt_tokens] [2]
    2020-08-05 02:23:26,089 INFO - *** src/amf/amfnd/avnd_comp.h:480: Complex multi-line /.../-style comment found. Lint may give bogus warnings. Consider replacing these with //-style comments, with #if 0...#endif, or with more clearly structured multi-line comments. [readability/multiline_comment] [5]
    2020-08-05 02:23:26,089 INFO - *** src/amf/amfnd/avnd_comp.h:481: Use operator || instead of or [readability/alt_tokens] [2]
    2020-08-05 02:23:26,089 INFO - *** src/amf/amfnd/avnd_comp.h:508: Lines should be <= 80 characters long [whitespace/line_length] [2]
    2020-08-05 02:23:26,089 INFO - *** src/amf/amfnd/avnd_comp.h:509: Lines should be <= 80 characters long [whitespace/line_length] [2]
    2020-08-05 02:23:26,089 INFO - *** src/amf/amfnd/avnd_comp.h:630: Using C-style cast. Use reinterpret_cast<uint8_t *="">(...) instead [readability/casting] [4]
    2020-08-05 02:23:26,090 INFO - *** src/amf/amfnd/avnd_comp.h:632: Using C-style cast. Use reinterpret_cast<avnd_comp *="">(...) instead [readability/casting] [4]
    2020-08-05 02:23:26,090 INFO - *** src/amf/amfnd/avnd_comp.h:636: Using C-style cast. Use reinterpret_cast<uint8_t *="">(...) instead [readability/casting] [4]
    2020-08-05 02:23:26,090 INFO - *** src/amf/amfnd/avnd_comp.h:637: Using C-style cast. Use reinterpret_cast<uint8_t *="">(...) instead [readability/casting] [4]
    2020-08-05 02:23:26,090 INFO - *** src/amf/amfnd/avnd_comp.h:637: Weird number of spaces at line-start. Are you using a 2-space indent? [whitespace/indent] [3]
    2020-08-05 02:23:26,090 INFO - *** src/amf/amfnd/avnd_comp.h:639: Using C-style cast. Use reinterpret_cast<avnd_comp_csi_rec *="">(...) instead [readability/casting] [4]
    2020-08-05 02:23:26,090 INFO - *** src/amf/amfnd/avnd_comp.h:646: Using C-style cast. Use reinterpret_cast<uint8_t *="">(...) instead [readability/casting] [4]
    2020-08-05 02:23:26,091 INFO - *** src/amf/amfnd/avnd_comp.h:657: Using C-style cast. Use reinterpret_cast<uint8_t *="">(...) instead [readability/casting] [4]
    2020-08-05 02:23:26,091 INFO - *** src/amf/amfnd/avnd_comp.h:667: Using C-style cast. Use reinterpret_cast<uint8_t *="">(...) instead [readability/casting] [4]
    2020-08-05 02:23:26,091 INFO - *** src/amf/amfnd/avnd_comp.h:677: Using C-style cast. Use reinterpret_cast<avnd_comp_hc_rec *="">(...) instead [readability/casting] [4]
    2020-08-05 02:23:26,091 INFO - *** src/amf/amfnd/avnd_comp.h:678: Using C-style cast. Use reinterpret_cast<uint8_t *="">(...) instead [readability/casting] [4]
    2020-08-05 02:23:26,091 INFO - *** src/amf/amfnd/avnd_comp.h:692: Is this a non-const reference? If so, make const or use a pointer: uint32_t &o_rc [runtime/references] [2]
    2020-08-05 02:23:26,091 INFO - *** src/amf/amfnd/avnd_comp.h:699: Almost always, snprintf is better than strcpy [runtime/printf] [4]
    2020-08-05 02:23:26,092 INFO - *** src/amf/amfnd/avnd_comp.h:701: Consider using strtok_r(...) instead of strtok(...) for improved thread safety. [runtime/threadsafe_fn] [2]
    2020-08-05 02:23:26,092 INFO - *** src/amf/amfnd/avnd_comp.h:706: Consider using strtok_r(...) instead of strtok(...) for improved thread safety. [runtime/threadsafe_fn] [2]
    2020-08-05 02:23:26,092 INFO - *** src/amf/amfnd/avnd_comp.h:709: Almost always, snprintf is better than strcpy [runtime/printf] [4]
    2020-08-05 02:23:26,092 INFO - *** src/amf/amfnd/avnd_comp.h:714: Use int16/int64/etc, rather than the C type long [runtime/int] [4]
    2020-08-05 02:23:26,092 INFO - *** src/amf/amfnd/avnd_comp.h:714: Using C-style cast. Use static_cast<char>(...) instead [readability/casting] [4]
    2020-08-05 02:23:26,092 INFO - *** src/amf/amfnd/avnd_comp.h:717: If an else has a brace on one side, it should have it on both [readability/braces] [5]
    2020-08-05 02:23:26,092 INFO - *** src/amf/amfnd/avnd_comp.h:732: Using C-style cast. Use reinterpret_cast<uint32_t *="">(...) instead [readability/casting] [4]
    2020-08-05 02:23:26,092 INFO - *** src/amf/amfnd/avnd_comp.h:799: If an else has a brace on one side, it should have it on both [readability/braces] [5]
    2020-08-05 02:23:26,092 INFO - *** src/amf/amfnd/avnd_comp.h:814: If an else has a brace on one side, it should have it on both [readability/braces] [5]
    2020-08-05 02:23:26,092 INFO - *** src/amf/amfnd/avnd_comp.h:821: Is this a non-const reference? If so, make const or use a pointer: uint32_t& o_found [runtime/references] [2]
    2020-08-05 02:23:26,093 INFO - *** src/amf/amfnd/avnd_comp.h:826: Line contains only semicolon. If this should be an empty statement, use {} instead. [whitespace/semicolon] [5]
    2020-08-05 02:23:26,093 INFO - *** src/amf/amfnd/clm.cc:245: Lines should be <= 80 characters long [whitespace/line_length] [2]
    2020-08-05 02:23:26,093 INFO - *** src/amf/amfnd/clm.cc:249: Lines should be <= 80 characters long [whitespace/line_length] [2]
    2020-08-05 02:23:26,093 INFO - *** src/amf/amfnd/clm.cc:251: Lines should be <= 80 characters long [whitespace/line_length] [2]
    2020-08-05 02:23:26,093 INFO - *** src/amf/amfnd/clm.cc:254: At least two spaces is best between code and comments [whitespace/comments] [2]
    2020-08-05 02:23:26,093 INFO - *** src/amf/amfnd/clm.cc:255: At least two spaces is best between code and comments [whitespace/comments] [2]
    2020-08-05 02:23:26,093 INFO - *** src/amf/amfnd/clm.cc:256: At least two spaces is best between code and comments [whitespace/comments] [2]
    2020-08-05 02:23:26,093 INFO - *** src/amf/amfnd/clm.cc:257: At least two spaces is best between code and comments [whitespace/comments] [2]
    2020-08-05 02:23:26,093 INFO - *** src/amf/amfnd/clm.cc:258: At least two spaces is best between code and comments [whitespace/comments] [2]
    2020-08-05 02:23:26,093 INFO - *** src/amf/amfnd/clm.cc:285: If an else has a brace on one side, it should have it on both [readability/braces] [5]
    2020-08-05 02:23:26,094 INFO - *** src/amf/amfnd/err.cc:1222: Redundant blank line at the end of a code block should be deleted. [whitespace/blank_line] [3]
    2020-08-05 02:23:26,094 INFO - *** src/amf/amfnd/err.cc:1328: If an else has a brace on one side, it should have it on both [readability/braces] [5]
    2020-08-05 02:23:26,094 INFO - *** src/amf/amfnd/err.cc:185: Using C-style cast. Use reinterpret_cast<void *="">(...) instead [readability/casting] [4]
    2020-08-05 02:23:26,094 INFO - *** src/amf/amfnd/err.cc:190: If/else bodies with multiple statements require braces [readability/braces] [4]
    2020-08-05 02:23:26,094 INFO - *** src/amf/amfnd/err.cc:192: If an else has a brace on one side, it should have it on both [readability/braces] [5]
    2020-08-05 02:23:26,094 INFO - *** src/amf/amfnd/err.cc:406: Lines should be <= 80 characters long [whitespace/line_length] [2]
    2020-08-05 02:23:26,094 INFO - *** src/amf/amfnd/err.cc:471: If an else has a brace on one side, it should have it on both [readability/braces] [5]
    2020-08-05 02:23:26,094 INFO - *** src/amf/amfnd/err.cc:745: If an else has a brace on one side, it should have it on both [readability/braces] [5]
    2020-08-05 02:23:26,095 INFO - *** src/amf/amfnd/err.cc:905: Missing username in TODO; it should look like "// TODO(my_username): Stuff." [readability/todo] [2]
    2020-08-05 02:23:26,095 INFO - *** src/amf/amfnd/imm.cc:110: Missing space before ( in if( [whitespace/parens] [5]
    2020-08-05 02:23:26,095 INFO - *** src/amf/amfnd/imm.cc:159: Add #include <string> for string [build/include_what_you_use] [4]
    2020-08-05 02:23:26,095 INFO - *** src/amf/amfnd/imm.cc:168: Lines should be <= 80 characters long [whitespace/line_length] [2]
    2020-08-05 02:23:26,095 INFO - *** src/amf/amfnd/imm.cc:192: Using C-style cast. Use reinterpret_cast<saimmattrvaluest_2 ***="">(...) instead [readability/casting] [4]
    2020-08-05 02:23:26,095 INFO - *** src/amf/amfnd/imm.cc:213: Using C-style cast. Use reinterpret_cast<saimmattrvaluest_2 ***="">(...) instead [readability/casting] [4]
    2020-08-05 02:23:26,095 INFO - *** src/amf/amfnd/imm.cc:229: If statement had no body and no else clause [whitespace/empty_if_body] [4]
    2020-08-05 02:23:26,095 INFO - *** src/amf/amfnd/imm.cc:236: Using C-style cast. Use reinterpret_cast<saimmattrvaluest_2 ***="">(...) instead [readability/casting] [4]
    2020-08-05 02:23:26,095 INFO - *** src/amf/amfnd/imm.cc:238: Lines should be <= 80 characters long [whitespace/line_length] [2]
    2020-08-05 02:23:26,095 INFO - *** src/amf/amfnd/imm.cc:253: If statement had no body and no else clause [whitespace/empty_if_body] [4]
    2020-08-05 02:23:26,095 INFO - *** src/amf/amfnd/imm.cc:258: If statement had no body and no else clause [whitespace/empty_if_body] [4]
    2020-08-05 02:23:26,095 INFO - *** src/amf/amfnd/imm.cc:266: Using C-style cast. Use reinterpret_cast<saimmattrvaluest_2 ***="">(...) instead [readability/casting] [4]
    2020-08-05 02:23:26,096 INFO - *** src/amf/amfnd/imm.cc:284: If statement had no body and no else clause [whitespace/empty_if_body] [4]
    2020-08-05 02:23:26,096 INFO - *** src/amf/amfnd/imm.cc:290: If statement had no body and no else clause [whitespace/empty_if_body] [4]
    2020-08-05 02:23:26,096 INFO - *** src/amf/amfnd/imm.cc:302: Lines should be <= 80 characters long [whitespace/line_length] [2]
    2020-08-05 02:23:26,096 INFO - *** src/amf/amfnd/imm.cc:302: Using C-style cast. Use reinterpret_cast<uint8_t *="">(...) instead [readability/casting] [4]
    2020-08-05 02:23:26,096 INFO - *** src/amf/amfnd/imm.cc:337: Lines should be <= 80 characters long [whitespace/line_length] [2]
    2020-08-05 02:23:26,096 INFO - *** src/amf/amfnd/imm.cc:77: Using C-style cast. Use reinterpret_cast<avnd_evt *="">(...) instead [readability/casting] [4]</avnd_evt></uint8_t></saimmattrvaluest_2></saimmattrvaluest_2></saimmattrvaluest_2></saimmattrvaluest_2></string></void></uint32_t></char></uint8_t></avnd_comp_hc_rec></uint8_t></uint8_t></uint8_t></avnd_comp_csi_rec></uint8_t></uint8_t></avnd_comp></uint8_t></string></std::string,></std::string,></string></uint8_t>

     

  • Minh Hon Chau

    Minh Hon Chau - 2020-08-05

    2020-08-05 02:19:34,433 INFO - *** cppcheck errors removed by patch: 2
    2020-08-05 02:19:34,434 INFO - *** cppcheck errors added by patch: 5
    2020-08-05 02:19:34,434 INFO - *** List of cppcheck errors removed by patch ***
    2020-08-05 02:19:34,434 INFO - *** [src/amf/amfnd/err.cc:1139] -> [src/amf/amfnd/err.cc:1138]: (style) Variable 'rc' is reassigned a value before the old one has been used.
    2020-08-05 02:19:34,434 INFO - *** [src/amf/amfnd/err.cc:670] -> [src/amf/amfnd/err.cc:667]: (warning) Variable 'rc' is reassigned a value before the old one has been used. 'break;' missing?
    2020-08-05 02:19:34,434 INFO - *** List of cppcheck errors added by patch ***
    2020-08-05 02:19:34,434 INFO - *** [src/amf/amfnd/clm.cc:237]: (style) Variable 'name' is assigned a value that is never used.
    2020-08-05 02:19:34,434 INFO - *** [src/amf/amfnd/err.cc:1154] -> [src/amf/amfnd/err.cc:1153]: (style) Variable 'rc' is reassigned a value before the old one has been used.
    2020-08-05 02:19:34,434 INFO - *** [src/amf/amfnd/err.cc:186] -> [src/amf/amfnd/err.cc:190]: (warning) Either the condition 'if(evt_er)' is redundant or there is possible null pointer dereference: evt_er.
    2020-08-05 02:19:34,434 INFO - *** [src/amf/amfnd/err.cc:188] -> [src/amf/amfnd/err.cc:190]: (warning) Either the condition 'if(evt_er)' is redundant or there is possible null pointer dereference: evt_er.
    2020-08-05 02:19:34,434 INFO - *** [src/amf/amfnd/err.cc:685] -> [src/amf/amfnd/err.cc:682]: (warning) Variable 'rc' is reassigned a value before the old one has been used. 'break;' missing?

     

  • Minh Hon Chau

    Minh Hon Chau - 2020-08-09
    • status: review --> fixed
    • assigned_to: Anand Sundararaj --> nobody
     

  • Minh Hon Chau

    Minh Hon Chau - 2020-08-09

    commit 64b48fbde5000ab7b57fd43d63911be64047a628
    Author: Anand Sundararaj s.anand@GetHighAvailability.com
    Date: Sun Aug 9 11:48:43 2020 +1000

    amf: support error report on non local component [#109] V3
     

Log in to post a comment.

MongoDB Logo MongoDB