Ticket #348 (closed defect: fixed)

Opened 7 years ago

Last modified 7 years ago

collectd crash

Reported by: jesus Assigned to: jesus
Priority: major Milestone:
Component: noitd Severity: annoying
Keywords: Cc:

Description

noit config:

  <module image="collectd" name="collectd">
     <config>
       <username>xxxx</username>
       <password>yyyy</password>
       <security_level>1</security_level>
     </config>
   </module>

valgrind:

==10492== Memcheck, a memory error detector
==10492== Copyright (C) 2002-2009, and GNU GPL'd, by Julian Seward et al.
==10492== Using Valgrind-3.6.0.SVN-Debian and LibVEX; rerun with -h
for copyright info
==10492== Command: noitd -c /etc/reconnoiter/noit.conf -D
==10492==
[2011-02-03 14:34:40.359677] Processed 1 includes
[2011-02-03 14:34:40.696829] Found 29 /noit/logs//log stanzas
[2011-02-03 14:34:40.772376] Found 1 outlets for log 'error'
[2011-02-03 14:34:41.651330] rlim { 1014, 1014 }
[2011-02-03 14:34:42.344006] Found 1 acl stanzas
[2011-02-03 14:34:42.631835] Module selfcheck successfully loaded.
[2011-02-03 14:34:42.844569] Module ping_icmp successfully loaded.
[2011-02-03 14:34:42.965947] Module dns successfully loaded.
[2011-02-03 14:34:46.770346] Module snmp successfully loaded.
[2011-02-03 14:34:46.829830] Module ssh2 successfully loaded.
[2011-02-03 14:34:46.852001] Module collectd successfully loaded.
[2011-02-03 14:34:48.012957] Module varnish successfully loaded.
[2011-02-03 14:34:48.331806] Module http successfully loaded.
[2011-02-03 14:34:48.442841] Module resmon successfully loaded.
[2011-02-03 14:34:48.512720] Module smtp successfully loaded.
[2011-02-03 14:34:48.625088] Module jmx successfully loaded.
[2011-02-03 14:34:48.969805] Found 4 /noit/listeners//listener stanzas
==10492== Invalid read of size 1
==10492==    at 0x4026038: strlen (mc_replace_strmem.c:282)
==10492==    by 0x7F64517: parse_part_sign_sha256 (collectd.c:737)
==10492==    by 0x7F62249: parse_packet (collectd.c:972)
==10492==    by 0x7F635BF: noit_collectd_handler (collectd.c:1375)
==10492==    by 0x8085177: eventer_epoll_impl_trigger (eventer_epoll_impl.c:201)
==10492==    by 0x8085388: eventer_epoll_impl_loop (eventer_epoll_impl.c:266)
==10492==    by 0x805966A: child_main (noitd.c:264)
==10492==    by 0x805A294: main (noitd.c:347)
==10492==  Address 0x0 is not stack'd, malloc'd or (recently) free'd
==10492==
==10492==
==10492== Process terminating with default action of signal 11 (SIGSEGV)
==10492==  Access not within mapped region at address 0x0
==10492==    at 0x4026038: strlen (mc_replace_strmem.c:282)
==10492==    by 0x7F64517: parse_part_sign_sha256 (collectd.c:737)
==10492==    by 0x7F62249: parse_packet (collectd.c:972)
==10492==    by 0x7F635BF: noit_collectd_handler (collectd.c:1375)
==10492==    by 0x8085177: eventer_epoll_impl_trigger (eventer_epoll_impl.c:201)
==10492==    by 0x8085388: eventer_epoll_impl_loop (eventer_epoll_impl.c:266)
==10492==    by 0x805966A: child_main (noitd.c:264)
==10492==    by 0x805A294: main (noitd.c:347)
==10492==  If you believe this happened as a result of a stack
==10492==  overflow in your program's main thread (unlikely but
==10492==  possible), you can try to increase the size of the
==10492==  main thread stack using the --main-stacksize= flag.
==10492==  The main thread stack size used in this run was 8388608.
==10492==
==10492== HEAP SUMMARY:
==10492==     in use at exit: 716,386 bytes in 17,625 blocks
==10492==   total heap usage: 65,443 allocs, 47,818 frees, 6,905,714
bytes allocated
==10492==
==10492== LEAK SUMMARY:
==10492==    definitely lost: 68 bytes in 3 blocks
==10492==    indirectly lost: 0 bytes in 0 blocks
==10492==      possibly lost: 1,512 bytes in 15 blocks
==10492==    still reachable: 714,806 bytes in 17,607 blocks
==10492==         suppressed: 0 bytes in 0 blocks
==10492== Rerun with --leak-check=full to see details of leaked memory
==10492==
==10492== For counts of detected and suppressed errors, rerun with: -v
==10492== ERROR SUMMARY: 1 errors from 1 contexts (suppressed: 72 from 10)
Killed

Change History

02/04/11 00:38:26 changed by jesus

  • status changed from new to closed.
  • resolution set to fixed.

(In [1544]) I believe this fixes #348