1
0
mirror of https://github.com/ericonr/sndio.git synced 2024-02-18 04:45:21 -06:00
sndio/sndiod/miofile.c
Alexandre Ratchov f29a76bc19 On error, drop clients and close the device only if it's still open
Fixes crashes when USB devices are disconnected, caused by an attempt
to close the already closed device: it was closed once when its ref
counter drops to zero (after the last client is disconnected) and once
with an explicit call to dev_close() on the error code-path.

The same pattern was used for MIDI ports, it's fixed as well.
2020-06-12 17:26:38 +02:00

192 lines
3.7 KiB
C

/* $OpenBSD$ */
/*
* Copyright (c) 2008-2012 Alexandre Ratchov <alex@caoua.org>
*
* Permission to use, copy, modify, and distribute this software for any
* purpose with or without fee is hereby granted, provided that the above
* copyright notice and this permission notice appear in all copies.
*
* THE SOFTWARE IS PROVIDED "AS IS" AND THE AUTHOR DISCLAIMS ALL WARRANTIES
* WITH REGARD TO THIS SOFTWARE INCLUDING ALL IMPLIED WARRANTIES OF
* MERCHANTABILITY AND FITNESS. IN NO EVENT SHALL THE AUTHOR BE LIABLE FOR
* ANY SPECIAL, DIRECT, INDIRECT, OR CONSEQUENTIAL DAMAGES OR ANY DAMAGES
* WHATSOEVER RESULTING FROM LOSS OF USE, DATA OR PROFITS, WHETHER IN AN
* ACTION OF CONTRACT, NEGLIGENCE OR OTHER TORTIOUS ACTION, ARISING OUT OF
* OR IN CONNECTION WITH THE USE OR PERFORMANCE OF THIS SOFTWARE.
*/
#include <sys/types.h>
#include <sys/time.h>
#include <poll.h>
#include <stdio.h>
#include <stdlib.h>
#include <string.h>
#include <sndio.h>
#include "defs.h"
#include "file.h"
#include "midi.h"
#include "miofile.h"
#include "utils.h"
int port_mio_pollfd(void *, struct pollfd *);
int port_mio_revents(void *, struct pollfd *);
void port_mio_in(void *);
void port_mio_out(void *);
void port_mio_hup(void *);
struct fileops port_mio_ops = {
"mio",
port_mio_pollfd,
port_mio_revents,
port_mio_in,
port_mio_out,
port_mio_hup
};
/*
* open the port using one of the provided paths
*/
static struct mio_hdl *
port_mio_openlist(struct port *c, unsigned int mode)
{
struct mio_hdl *hdl;
struct name *n;
int idx;
idx = 0;
n = c->path_list;
while (1) {
if (n == NULL)
break;
hdl = mio_open(n->str, mode, 1);
if (hdl != NULL) {
if (log_level >= 2) {
port_log(c);
log_puts(": using ");
log_puts(n->str);
log_puts("\n");
}
return hdl;
}
n = n->next;
idx++;
}
return NULL;
}
int
port_mio_open(struct port *p)
{
p->mio.hdl = port_mio_openlist(p, p->midi->mode);
if (p->mio.hdl == NULL)
return 0;
p->mio.file = file_new(&port_mio_ops, p, "port", mio_nfds(p->mio.hdl));
return 1;
}
/*
* Open an alternate port. Upon success, close the old port
* and continue using the new one.
*/
int
port_mio_reopen(struct port *p)
{
struct mio_hdl *hdl;
hdl = port_mio_openlist(p, p->midi->mode);
if (hdl == NULL) {
if (log_level >= 1) {
port_log(p);
log_puts(": couldn't open an alternate port\n");
}
return 0;
}
/* close unused device */
file_del(p->mio.file);
mio_close(p->mio.hdl);
p->mio.hdl = hdl;
p->mio.file = file_new(&port_mio_ops, p, "port", mio_nfds(hdl));
return 1;
}
void
port_mio_close(struct port *p)
{
file_del(p->mio.file);
mio_close(p->mio.hdl);
}
int
port_mio_pollfd(void *addr, struct pollfd *pfd)
{
struct port *p = addr;
struct midi *ep = p->midi;
int events = 0;
if (ep->mode & MODE_MIDIIN)
events |= POLLIN;
if ((ep->mode & MODE_MIDIOUT) && ep->obuf.used > 0)
events |= POLLOUT;
return mio_pollfd(p->mio.hdl, pfd, events);
}
int
port_mio_revents(void *addr, struct pollfd *pfd)
{
struct port *p = addr;
return mio_revents(p->mio.hdl, pfd);
}
void
port_mio_in(void *arg)
{
unsigned char data[MIDI_BUFSZ];
struct port *p = arg;
struct midi *ep = p->midi;
int n;
for (;;) {
n = mio_read(p->mio.hdl, data, MIDI_BUFSZ);
if (n == 0)
break;
midi_in(ep, data, n);
}
}
void
port_mio_out(void *arg)
{
struct port *p = arg;
struct midi *ep = p->midi;
unsigned char *data;
int n, count;
for (;;) {
data = abuf_rgetblk(&ep->obuf, &count);
if (count == 0)
break;
n = mio_write(p->mio.hdl, data, count);
if (n == 0)
break;
abuf_rdiscard(&ep->obuf, n);
if (n < count)
break;
}
if (p->state == PORT_DRAIN && ep->obuf.used == 0)
port_close(p);
midi_fill(ep);
}
void
port_mio_hup(void *arg)
{
struct port *p = arg;
if (!port_reopen(p))
port_abort(p);
}