yukijoou/kwin
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

plugins/screencast: Avoid closing dmabuf fds twice

Browse source 
dmabuf fds are owned by the GraphicsBuffer, which will be destroyed when
`m_dmabufDataForPwBuffer.remove(buffer);` is executed.
This commit is contained in:
Vlad Zahorodnii 2024-03-18 15:28:24 +02:00
parent 0897dbff75
commit fb7dd07076
1 changed files with 0 additions and 4 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

4
src/plugins/screencast/screencaststream.cpp
View file

@ -292,10 +292,6 @@ void ScreenCastStream::onStreamRemoveBuffer(pw_buffer *buffer)
if (spa_data && spa_data->type == SPA_DATA_MemFd) { if (spa_data && spa_data->type == SPA_DATA_MemFd) {
munmap(spa_data->data, spa_data->maxsize); munmap(spa_data->data, spa_data->maxsize);
close(spa_data->fd); close(spa_data->fd);
} else if (spa_data && spa_data->type == SPA_DATA_DmaBuf) {
for (int i = 0, c = buffer->buffer->n_datas; i < c; ++i) {
close(buffer->buffer->datas[i].fd);
}
} }
} }