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
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
case it is; see the original report), even if we had
_malloc( )ed d->address, it'd still do the wrong thing
Karmic but hopefully for -updates
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
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
06:03 < dtchen> pitti: yes, I'll update the original bug report