On Tue, 2016-08-16 at 16:06 +0200, Peter Meerwald-Stadler wrote: > From: Peter Meerwald <[email protected]> > > CID 1533121 > > Signed-off-by: Peter Meerwald-Stadler <[email protected]> > --- > src/modules/bluetooth/backend-native.c | 6 ++++-- > 1 file changed, 4 insertions(+), 2 deletions(-) > > diff --git a/src/modules/bluetooth/backend-native.c > b/src/modules/bluetooth/backend-native.c > index 86376c0..b5f78f5 100644 > --- a/src/modules/bluetooth/backend-native.c > +++ b/src/modules/bluetooth/backend-native.c > @@ -232,13 +232,16 @@ static void rfcomm_io_callback(pa_mainloop_api > *io, pa_io_event *e, int fd, pa_i > int gain; > > len = read(fd, buf, 511); > + if (len < 0) { > + pa_log_error("RFCOMM read error: %s", > pa_cstrerror(errno)); > + goto fail; > + }
This is certainly an improvement. Maybe we should use pa_read() instead of read() to handle EINTR, though? -- Tanu _______________________________________________ pulseaudio-discuss mailing list [email protected] https://lists.freedesktop.org/mailman/listinfo/pulseaudio-discuss
