You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@avro.apache.org by "Lucas Martin-King (JIRA)" <ji...@apache.org> on 2012/09/14 03:50:07 UTC

[jira] [Commented] (AVRO-1161) Avro-C: Memory Leak in avro_schema_record()

    [ https://issues.apache.org/jira/browse/AVRO-1161?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13455528#comment-13455528 ] 

Lucas Martin-King commented on AVRO-1161:
-----------------------------------------

Upon further investigation, this is merely a side effect of the fact that somewhere within datafile.c or schema.c, an avro_schema_decref() is not being performed.

If I add an additional avro_schema_decref() to avro_file_reader_close(), then the leak is resolved, as the refcount will be zero the on the 2nd call, thus freeing up the schema. However, this is the band-aid solution, and I'm going to try and find where the decref call is missing from.
                
> Avro-C: Memory Leak in avro_schema_record()
> -------------------------------------------
>
>                 Key: AVRO-1161
>                 URL: https://issues.apache.org/jira/browse/AVRO-1161
>             Project: Avro
>          Issue Type: Bug
>          Components: c
>    Affects Versions: 1.7.1
>            Reporter: Lucas Martin-King
>              Labels: memory_leak
>
> There seems to be a bug in the use of avro_schema_record() within the datafile reader. I'm currently investigating this myself, and can hopefully submit a patch soon, however I am wondering if there is anything I am forgetting to do when using the Avro API?
> Our program is opening and closing lots of log files, and Valgrind gives me this message:
> ==20151== 32,518 (48 direct, 32,470 indirect) bytes in 1 blocks are definitely lost in loss record 62 of 65
> ==20151==    at 0x4C2BFCB: malloc (in /usr/lib/valgrind/vgpreload_memcheck-amd64-linux.so)
> ==20151==    by 0x4C2C27F: realloc (in /usr/lib/valgrind/vgpreload_memcheck-amd64-linux.so)
> ==20151==    by 0x4E3D074: avro_default_allocator (allocation.c:36)
> ==20151==    by 0x4E60F8C: avro_schema_record (schema.c:558)
> ==20151==    by 0x4E61A5E: avro_schema_from_json_t (schema.c:856)
> ==20151==    by 0x4E622E9: avro_schema_from_json_root (schema.c:1083)
> ==20151==    by 0x4E624C9: avro_schema_from_json_length (schema.c:1127)
> ==20151==    by 0x4E3FE92: file_read_header (datafile.c:314)
> ==20151==    by 0x4E40796: avro_file_reader_fp (datafile.c:491)
> ==20151==    by 0x4082D3: LogReader::LogReader(_IO_FILE*, char const*) (LogReader.cpp:32)
> ==20151==    by 0x4066A3: main (main.cpp:206)
> A similar report can be reproduced when using avrocat on a single avro file.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira