Menu

#3331 Valgrind reports errors

5.23.03
fixed
None
defect
unknown
d
5.23.03
minor
False
2023-03-27
2023-03-01
No

Valgrind detected errors:

==484== 542 (536 direct, 6 indirect) bytes in 1 blocks are definitely lost in loss record 312 of 368
==484== at 0x4C3217F: operator new(unsigned long) (in /usr/lib/valgrind/vgpreload_memcheck-amd64-linux.so)
==484== by 0x1636BC: csiattr_create(std::__cxx11::basic_string<char, std::char_traits\<char="">, std::allocator\<char> > const&, SaImmAttrValuesT_2 const) (csiattr.cc:78)
==484== by 0x164443: csiattr_create_apply (csiattr.cc:519)
==484== by 0x164443: csiattr_ccb_apply_cb(CcbUtilOperationData
) (csiattr.cc:713)
==484== by 0x172155: ccb_apply_cb(unsigned long long, unsigned long long) (imm.cc:1265)
==484== by 0x54B0C94: imma_process_callback_info(imma_cb
, imma_client_node, imma_callback_info, unsigned long long) </char></char,>

==407== Invalid read of size 1
==407== at 0x5732C3A: mds_svc_op_uninstall (mds_svc_op.c:375)
==407== by 0x57320C7: ncsmds_api (mds_papi.c:147)
==407== by 0x54A31D2: imma_mds_unregister(imma_cb*) (imma_mds.cc:171)
==407== by 0x54A25D4: imma_destroy (imma_init.cc:219)
==407== by 0x54A25D4: imma_shutdown(ncsmds_svc_id) (imma_init.cc:337)
==407== by 0x54AF316: saImmOmFinalize (imma_om_api.cc:940)
==407== by 0x5061961: immutil_saImmOmFinalize (immutil.c:1572)
==407== by 0x141267: hydra_config_get (main.cc:775)
==407== by 0x141267: avnd_cb_create (main.cc:349)

==461== Mismatched free() / delete / delete []
==461== at 0x4C3323B: operator delete(void) (in /usr/lib/valgrind/vgpreload_memcheck-amd64-linux.so)
==461== by 0x1308D4: comp_init(avnd_comp_tag
, SaImmAttrValuesT_2 const) (compdb.cc:1422)
==461== by 0x131066: avnd_comp_config_reinit(avnd_comp_tag
) (compdb.cc:1759)
==461== by 0x123FD7: avnd_comp_clc_uninst_inst_hdler(avnd_cb_tag
, avnd_comp_tag) (clc.cc:1584)
==461== by 0x124390: avnd_comp_clc_fsm_run(avnd_cb_tag
, avnd_comp_tag, avnd_comp_clc_pres_fsm_ev) (clc.cc:887)
==461== by 0x153FE6: avnd_su_pres_uninst_suinst_hdler(avnd_cb_tag
, avnd_su_tag, avnd_comp_tag) (susm.cc:2145)
==461== by 0x1567C0: avnd_su_pres_fsm_run(avnd_cb_tag, avnd_su_tag, avnd_comp_tag, avnd_su_pres_fsm_ev) (susm.cc:1604)
==461== by 0x15C3AA: avnd_evt_ir_evh(avnd_cb_tag
, avnd_evt_tag*) (susm.cc:4236)
==461== by 0x141D25: avnd_evt_process (main.cc:692)
==461== by 0x141D25: avnd_main_process() (main.cc:644)
==461== by 0x1170AD: main (main.cc:225)

==407== Invalid read of size 8
==407== at 0x119942: avnd_evt_tmr_cbk_resp_evh(avnd_cb_tag, avnd_evt_tag) (cbq.cc:678)
==407== by 0x141D15: avnd_evt_process (main.cc:692)
==407== by 0x141D15: avnd_main_process() (main.cc:644)
==407== by 0x1170AD: main (main.cc:225)
==407== Address 0x8bb2ad0 is 64 bytes inside a block of size 112 free'd
==407== at 0x4C3323B: operator delete(void) (in /usr/lib/valgrind/vgpreload_memcheck-amd64-linux.so)
==407== by 0x11962B: avnd_comp_cbq_rec_pop_and_del(avnd_cb_tag
, avnd_comp_tag, unsigned int, bool) (cbq.cc:973)
==407== by 0x119941: avnd_evt_tmr_cbk_resp_evh(avnd_cb_tag
, avnd_evt_tag*) (cbq.cc:678)
==407== by 0x141D15: avnd_evt_process (main.cc:692)

==428== 8,072 (56 direct, 8,016 indirect) bytes in 1 blocks are definitely lost in loss record 285 of 289
==428== at 0x4C31B0F: malloc (in /usr/lib/valgrind/vgpreload_memcheck-amd64-linux.so)
==428== by 0x5914C2B: sysf_alloc_pkt (sysf_mem.c:429)
==428== by 0x5903A1F: ncs_enc_init_space_pp (hj_ubaid.c:144)
==428== by 0x5931996: mdtm_fill_data (mds_dt_common.c:1453)
==428== by 0x5932CC2: mdtm_process_recv_message_common (mds_dt_common.c:544)
==428== by 0x5933061: mdtm_process_recv_data (mds_dt_common.c:1125)
==428== by 0x59351D6: mds_mdtm_process_recvdata (mds_dt_trans.c:1217)
==428== by 0x5936426: mdtm_process_poll_recv_data_tcp (mds_dt_trans.c:903)
==428== by 0x593683E: mdtm_process_recv_events_tcp (mds_dt_trans.c:995)
==428== by 0x61196DA: start_thread (pthread_create.c:463)

Related

Wiki: ChangeLog-5.23.03

Discussion

  • PhanTranQuocDat

    PhanTranQuocDat - 2023-03-01
    • status: assigned --> review
     
  • PhanTranQuocDat

    PhanTranQuocDat - 2023-03-02
    • Description has changed:

    Diff:

    --- old
    +++ new
    @@ -18,6 +18,19 @@
     ==407==    by 0x5061961: immutil_saImmOmFinalize (immutil.c:1572)
     ==407==    by 0x141267: hydra_config_get (main.cc:775)
     ==407==    by 0x141267: avnd_cb_create (main.cc:349)
    +    
    +==461== Mismatched free() / delete / delete []
    +==461==    at 0x4C3323B: operator delete(void*) (in /usr/lib/valgrind/vgpreload_memcheck-amd64-linux.so)
    +==461==    by 0x1308D4: comp_init(avnd_comp_tag*, SaImmAttrValuesT_2 const**) (compdb.cc:1422)
    +==461==    by 0x131066: avnd_comp_config_reinit(avnd_comp_tag*) (compdb.cc:1759)
    +==461==    by 0x123FD7: avnd_comp_clc_uninst_inst_hdler(avnd_cb_tag*, avnd_comp_tag*) (clc.cc:1584)
    +==461==    by 0x124390: avnd_comp_clc_fsm_run(avnd_cb_tag*, avnd_comp_tag*, avnd_comp_clc_pres_fsm_ev) (clc.cc:887)
    +==461==    by 0x153FE6: avnd_su_pres_uninst_suinst_hdler(avnd_cb_tag*, avnd_su_tag*, avnd_comp_tag*) (susm.cc:2145)
    +==461==    by 0x1567C0: avnd_su_pres_fsm_run(avnd_cb_tag*, avnd_su_tag*, avnd_comp_tag*, avnd_su_pres_fsm_ev) (susm.cc:1604)
    +==461==    by 0x15C3AA: avnd_evt_ir_evh(avnd_cb_tag*, avnd_evt_tag*) (susm.cc:4236)
    +==461==    by 0x141D25: avnd_evt_process (main.cc:692)
    +==461==    by 0x141D25: avnd_main_process() (main.cc:644)
    +==461==    by 0x1170AD: main (main.cc:225)
    
     ==407== Invalid read of size 8
     ==407==    at 0x119942: avnd_evt_tmr_cbk_resp_evh(avnd_cb_tag*, avnd_evt_tag*) (cbq.cc:678)
    
    • status: review --> assigned
     
  • PhanTranQuocDat

    PhanTranQuocDat - 2023-03-07
    • Description has changed:

    Diff:

    --- old
    +++ new
    @@ -42,3 +42,16 @@
     ==407==    by 0x11962B: avnd_comp_cbq_rec_pop_and_del(avnd_cb_tag*, avnd_comp_tag*, unsigned int, bool) (cbq.cc:973)
     ==407==    by 0x119941: avnd_evt_tmr_cbk_resp_evh(avnd_cb_tag*, avnd_evt_tag*) (cbq.cc:678)
     ==407==    by 0x141D15: avnd_evt_process (main.cc:692)
    +    
    +    
    +==428== 8,072 (56 direct, 8,016 indirect) bytes in 1 blocks are definitely lost in loss record 285 of 289
    +==428==    at 0x4C31B0F: malloc (in /usr/lib/valgrind/vgpreload_memcheck-amd64-linux.so)
    +==428==    by 0x5914C2B: sysf_alloc_pkt (sysf_mem.c:429)
    +==428==    by 0x5903A1F: ncs_enc_init_space_pp (hj_ubaid.c:144)
    +==428==    by 0x5931996: mdtm_fill_data (mds_dt_common.c:1453)
    +==428==    by 0x5932CC2: mdtm_process_recv_message_common (mds_dt_common.c:544)
    +==428==    by 0x5933061: mdtm_process_recv_data (mds_dt_common.c:1125)
    +==428==    by 0x59351D6: mds_mdtm_process_recvdata (mds_dt_trans.c:1217)
    +==428==    by 0x5936426: mdtm_process_poll_recv_data_tcp (mds_dt_trans.c:903)
    +==428==    by 0x593683E: mdtm_process_recv_events_tcp (mds_dt_trans.c:995)
    +==428==    by 0x61196DA: start_thread (pthread_create.c:463) 
    
     
  • PhanTranQuocDat

    PhanTranQuocDat - 2023-03-07
    • status: assigned --> review
     
  • PhanTranQuocDat

    PhanTranQuocDat - 2023-03-08

    commit b65c0887f7d9f240573b7067110cdccb03e79397 (HEAD -> develop, origin/develop)
    Author: dat.tq.phan dat.tq.phan@dektech.com.au
    Date: Wed Mar 1 09:53:44 2023 +0700

    amf: Fix errors reported by valgrind [#3331]
    
    Valgrind reports errors while run amf tests. This ticket
    fixes errors from the report.
    
     
  • PhanTranQuocDat

    PhanTranQuocDat - 2023-03-08
    • status: review --> fixed
     
  • PhanTranQuocDat

    PhanTranQuocDat - 2023-03-20
    • status: fixed --> assigned
     
  • PhanTranQuocDat

    PhanTranQuocDat - 2023-03-22
    • Part: - --> d
     
  • PhanTranQuocDat

    PhanTranQuocDat - 2023-03-22

    commit df7e4e8e33363ddcac19538370c7d29e768f7d4f (origin/develop)
    Author: dat.tq.phan dat.tq.phan@dektech.com.au
    Date: Mon Mar 20 10:47:19 2023 +0700

    amf: Fix errors reported by valgrind (cont) [#3331]
    
    This is the continous fix for valgrind report.
    The commit will fix memleak when trigger switch-over.
    
     
  • Gary Lee

    Gary Lee - 2023-03-27
    • status: assigned --> fixed
     

Log in to post a comment.