atom feed61 messages in at.iem.pd-listRe: [PD] fux_kinect
FromSent OnAttachments
sonia yuditskayaOct 25, 2011 6:07 pm 
sonia yuditskayaOct 25, 2011 6:13 pm 
vadeOct 25, 2011 7:00 pm 
Hans-Christoph SteinerOct 25, 2011 8:21 pm 
sonia yuditskayaOct 26, 2011 2:07 pm 
Hans-Christoph SteinerOct 26, 2011 2:54 pm 
sonia yuditskayaOct 26, 2011 3:27 pm 
Budi PrakosaOct 26, 2011 5:25 pm 
Hans-Christoph SteinerOct 26, 2011 6:00 pm 
Hans-Christoph SteinerOct 26, 2011 7:25 pm.patch, .patch
Budi PrakosaOct 26, 2011 7:49 pm 
Hans-Christoph SteinerOct 26, 2011 7:54 pm 
Budi PrakosaOct 26, 2011 10:23 pm 
Jordi SalaOct 27, 2011 4:39 am 
Budi PrakosaOct 27, 2011 4:50 am 
José Luis Santorcuato TapiaNov 11, 2011 4:19 am 
Hans-Christoph SteinerNov 11, 2011 7:30 am 
tim vetsNov 11, 2011 7:57 am 
Hans-Christoph SteinerNov 11, 2011 8:02 am 
tim vetsNov 11, 2011 8:06 am 
Mathieu BouchardNov 11, 2011 8:38 am 
tim vetsNov 11, 2011 8:55 am 
Hans-Christoph SteinerNov 11, 2011 9:10 am 
Mathieu BouchardNov 11, 2011 10:42 am 
tim vetsNov 11, 2011 10:49 am 
tim vetsNov 11, 2011 10:52 am 
tim vetsNov 11, 2011 11:04 am 
tim vetsNov 11, 2011 11:18 am 
tim vetsNov 11, 2011 11:24 am 
Hans-Christoph SteinerNov 11, 2011 12:25 pm 
Mathieu BouchardNov 11, 2011 12:25 pm 
tim vetsNov 11, 2011 12:54 pm.Other
tim vetsNov 11, 2011 12:58 pm 
Mathieu BouchardNov 12, 2011 8:53 am 
Mathieu BouchardNov 12, 2011 8:55 am 
Mathieu BouchardNov 12, 2011 9:01 am 
Mathieu BouchardNov 12, 2011 9:11 am 
Claude Heiland-AllenNov 12, 2011 9:13 am 
Mathieu BouchardNov 12, 2011 9:15 am 
tim vetsNov 12, 2011 9:19 am 
Mathieu BouchardNov 12, 2011 9:30 am 
Mathieu BouchardNov 12, 2011 9:33 am 
tim vetsNov 12, 2011 9:55 am 
tim vetsNov 12, 2011 9:58 am 
Mathieu BouchardNov 12, 2011 10:04 am 
tim vetsNov 12, 2011 10:08 am 
tim vetsNov 12, 2011 10:10 am.Other
tim vetsNov 14, 2011 5:44 am.txt
tim vetsNov 14, 2011 6:41 am 
Mathieu BouchardNov 14, 2011 7:53 am 
Budi PrakosaNov 14, 2011 9:03 am 
tim vetsNov 14, 2011 10:19 am 
Matthias KronlachnerNov 14, 2011 5:07 pm 
tim vetsNov 15, 2011 6:23 am 
tim vetsNov 15, 2011 7:11 am 
Mathieu BouchardNov 15, 2011 7:12 am 
Mathieu BouchardNov 15, 2011 7:14 am 
Matthias KronlachnerNov 15, 2011 11:26 am 
Budi PrakosaNov 21, 2011 9:24 pm 
Mathieu BouchardNov 22, 2011 7:18 am 
Budi PrakosaNov 22, 2011 4:03 pm 
Subject:Re: [PD] fux_kinect
From:Mathieu Bouchard (mat@artengine.ca)
Date:Nov 14, 2011 7:53:22 am
List:at.iem.pd-list

Le 2011-11-14 à 14:44:00, tim vets a écrit :

Attached is the output of "valgrind --leak-check=full pdextended" and opening fux_kinect-help.pd.

I think that you better not add --leak-check when just looking for a crash. But the only problem it does, is make the log file bigger.

Here's what I found (summarising the important error messages) :

Invalid write of size 1 at convert_bayer_to_rgb (in libfreenect) by [...] by libusb_handle_events_timeout (in libusb). Address 0xa066360 is [between 0 and 5] bytes after a block of size 307,200 alloc'd

This means that when libusb gives libfreenect the RGGB buffer and libfreenect is converting it to plain RGB, it makes a mistake and writes 6 bytes more than just 640*480 pixels, as if there were 2 extra pixels at the end. But I think that this is a bit misleading. It looks as if Valgrind was skipping a lot of other errors (probably by not able to detect them). Read on.

Then there is invalid write of size 1 from the same place but « Address is 749 bytes inside a block of size 12,800 free'd ». This doesn't look like any malloc that we know about. The number of bytes does not ring a bell either. But then it says that the memory was freed by request of /usr/lib/nvidia-current/libGL.so.270.41.06, which is a part of your video driver. (???)

But I just looked at how convert_bayer_to_rgb is written, and it doesn't look like it writes to more than one buffer. This function only writes 480 rows of 640 columns. But note that it writes RGB values, three bytes per pixel. That means you need a malloc(640*480*3) for each of the three RGB buffers in fux_kinect.