Comment 9 for bug 437293

Revision history for this message
Daniel T Chen (crimsun) wrote :

06:00 < dtchen> pitti: ok. firstly, it does the wrong thing with
                pa_sprintf_malloc(), including using the wrong parameter:
                address instead of d->address
06:01 < pitti> ah, so it even causes crashes/corruption?
06:01 < dtchen> pitti: secondly, in the case that d->address is bad (in this
                case it is; see the original report), even if we had
                _malloc()ed d->address, it'd still do the wrong thing
06:02 < dtchen> pitti: right, potentially it'd do even worse things
06:02 < pitti> dtchen: would you mind posting that to the original bug report,
               or adding a new one? to have a papertrail of the change?
06:02 < pitti> then it looks fine to me
06:03 < dtchen> pitti: so Lennart and I are discussing how to verify that the
                BT audio device is fully set up; it probably won't land for
                Karmic but hopefully for -updates
06:03 < dtchen> pitti: yes, I'll update the original bug report