don't take temporary copy of MIDI sequence
fixes potential boost shared_ptr double free.
This commit is contained in:
parent
2c8babf41e
commit
f499f4cfd2
|
@ -1467,10 +1467,10 @@ AlsaAudioBackend::main_process_thread ()
|
|||
i = 0;
|
||||
for (std::vector<AlsaPort*>::const_iterator it = _system_midi_out.begin (); it != _system_midi_out.end (); ++it, ++i) {
|
||||
assert (_rmidi_out.size() > i);
|
||||
const AlsaMidiBuffer src = static_cast<const AlsaMidiPort*>(*it)->const_buffer();
|
||||
const AlsaMidiBuffer * src = static_cast<const AlsaMidiPort*>(*it)->const_buffer();
|
||||
AlsaMidiOut *rm = _rmidi_out.at(i);
|
||||
rm->sync_time (clock1);
|
||||
for (AlsaMidiBuffer::const_iterator mit = src.begin (); mit != src.end (); ++mit) {
|
||||
for (AlsaMidiBuffer::const_iterator mit = src->begin (); mit != src->end (); ++mit) {
|
||||
rm->send_event ((*mit)->timestamp(), (*mit)->data(), (*mit)->size());
|
||||
}
|
||||
}
|
||||
|
@ -1826,8 +1826,8 @@ void* AlsaMidiPort::get_buffer (pframes_t /* nframes */)
|
|||
for (std::vector<AlsaPort*>::const_iterator i = get_connections ().begin ();
|
||||
i != get_connections ().end ();
|
||||
++i) {
|
||||
const AlsaMidiBuffer src = static_cast<const AlsaMidiPort*>(*i)->const_buffer ();
|
||||
for (AlsaMidiBuffer::const_iterator it = src.begin (); it != src.end (); ++it) {
|
||||
const AlsaMidiBuffer * src = static_cast<const AlsaMidiPort*>(*i)->const_buffer ();
|
||||
for (AlsaMidiBuffer::const_iterator it = src->begin (); it != src->end (); ++it) {
|
||||
(_buffer[_bufperiod]).push_back (boost::shared_ptr<AlsaMidiEvent>(new AlsaMidiEvent (**it)));
|
||||
}
|
||||
}
|
||||
|
|
|
@ -142,7 +142,7 @@ class AlsaMidiPort : public AlsaPort {
|
|||
DataType type () const { return DataType::MIDI; };
|
||||
|
||||
void* get_buffer (pframes_t nframes);
|
||||
const AlsaMidiBuffer const_buffer () const { return _buffer[_bufperiod]; }
|
||||
const AlsaMidiBuffer * const_buffer () const { return & _buffer[_bufperiod]; }
|
||||
|
||||
void next_period() { if (_n_periods > 1) { get_buffer(0); _bufperiod = (_bufperiod + 1) % _n_periods; } }
|
||||
void set_n_periods(int n) { if (n > 0 && n < 3) { _n_periods = n; } }
|
||||
|
|
|
@ -1769,10 +1769,10 @@ struct MidiEventSorter {
|
|||
}
|
||||
};
|
||||
|
||||
void DummyMidiPort::set_loopback (const DummyMidiBuffer src)
|
||||
void DummyMidiPort::set_loopback (DummyMidiBuffer const * const src)
|
||||
{
|
||||
_loopback.clear ();
|
||||
for (DummyMidiBuffer::const_iterator it = src.begin (); it != src.end (); ++it) {
|
||||
for (DummyMidiBuffer::const_iterator it = src->begin (); it != src->end (); ++it) {
|
||||
_loopback.push_back (boost::shared_ptr<DummyMidiEvent>(new DummyMidiEvent (**it)));
|
||||
}
|
||||
}
|
||||
|
@ -1838,8 +1838,8 @@ void* DummyMidiPort::get_buffer (pframes_t n_samples)
|
|||
if (source->is_physical() && source->is_terminal()) {
|
||||
source->get_buffer(n_samples); // generate signal.
|
||||
}
|
||||
const DummyMidiBuffer src = static_cast<const DummyMidiPort*>(*i)->const_buffer ();
|
||||
for (DummyMidiBuffer::const_iterator it = src.begin (); it != src.end (); ++it) {
|
||||
const DummyMidiBuffer *src = source->const_buffer ();
|
||||
for (DummyMidiBuffer::const_iterator it = src->begin (); it != src->end (); ++it) {
|
||||
_buffer.push_back (boost::shared_ptr<DummyMidiEvent>(new DummyMidiEvent (**it)));
|
||||
}
|
||||
}
|
||||
|
|
|
@ -193,10 +193,10 @@ class DummyMidiPort : public DummyPort {
|
|||
DataType type () const { return DataType::MIDI; };
|
||||
|
||||
void* get_buffer (pframes_t nframes);
|
||||
const DummyMidiBuffer const_buffer () const { return _buffer; }
|
||||
const DummyMidiBuffer * const_buffer () const { return &_buffer; }
|
||||
|
||||
void setup_generator (int, float const);
|
||||
void set_loopback (const DummyMidiBuffer src);
|
||||
void set_loopback (DummyMidiBuffer const * const src);
|
||||
|
||||
private:
|
||||
DummyMidiBuffer _buffer;
|
||||
|
|
Loading…
Reference in New Issue