After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 153264 - Leaks in sftp-method.c
Leaks in sftp-method.c
Status: RESOLVED FIXED
Product: gnome-vfs
Classification: Deprecated
Component: Module: ssh/sftp
unspecified
Other Linux
: Normal normal
: ---
Assigned To: gnome-vfs maintainers
gnome-vfs maintainers
Depends on:
Blocks:
 
 
Reported: 2004-09-21 10:42 UTC by Kjartan Maraas
Modified: 2005-02-11 22:18 UTC
See Also:
GNOME target: ---
GNOME version: 2.7/2.8


Attachments
patch for leak (1.05 KB, patch)
2004-09-21 10:43 UTC, Kjartan Maraas
none Details | Review
Updated version! (9.53 KB, patch)
2005-02-11 17:42 UTC, Christian Kellner
none Details | Review

Description Kjartan Maraas 2004-09-21 10:42:36 UTC
Valgrind reported this leak:

==28695== 4096 bytes in 1 blocks are definitely lost in loss record 159 of 185
==28695==    at 0x1B9054FA: realloc (vg_replace_malloc.c:197)
==28695==    by 0x1C315D9D: g_realloc (gmem.c:169)
==28695==    by 0x1E7ABFE9: buffer_check_alloc (sftp-method.c:250)
==28695==    by 0x1E7AC44D: buffer_recv (sftp-method.c:375)
==28695==    by 0x1E7AFE98: do_read_directory (sftp-method.c:2535)
==28695==    by 0x1C06F2F6: gnome_vfs_directory_read_next_cancellable
(gnome-vfs-directory.c:225)
==28695==    by 0x1C076D5B: _gnome_vfs_job_execute (gnome-vfs-job.c:1468)
==28695==    by 0x1C07515B: thread_routine (gnome-vfs-job-slave.c:64)
==28695==    by 0x1C085433: thread_entry (gnome-vfs-thread-pool.c:186)
==28695==    by 0x1C327E3B: g_thread_create_proxy (gthread.c:556)
==28695==    by 0x1C2C4925: thread_wrapper (vg_libpthread.c:867)
==28695==    by 0xB000F14F: do__quit (vg_scheduler.c:1872)
Comment 1 Kjartan Maraas 2004-09-21 10:43:33 UTC
Created attachment 31764 [details] [review]
patch for leak
Comment 2 Christian Kellner 2005-02-11 17:42:09 UTC
Created attachment 37358 [details] [review]
Updated version!
Comment 3 Christian Kellner 2005-02-11 22:18:36 UTC
I commited the patch above to cvs head!