render/dmabuf: cleanup on wlr_dmabuf_attributes_copy error

This commit is contained in:
Simon Ser 2021-02-02 19:52:20 +01:00
parent 975d14b799
commit 7ac2ce25e3

View file

@ -16,13 +16,22 @@ bool wlr_dmabuf_attributes_copy(struct wlr_dmabuf_attributes *dst,
struct wlr_dmabuf_attributes *src) { struct wlr_dmabuf_attributes *src) {
memcpy(dst, src, sizeof(struct wlr_dmabuf_attributes)); memcpy(dst, src, sizeof(struct wlr_dmabuf_attributes));
for (int i = 0; i < src->n_planes; ++i) { int i;
for (i = 0; i < src->n_planes; ++i) {
dst->fd[i] = fcntl(src->fd[i], F_DUPFD_CLOEXEC, 0); dst->fd[i] = fcntl(src->fd[i], F_DUPFD_CLOEXEC, 0);
if (dst->fd[i] < 0) { if (dst->fd[i] < 0) {
wlr_log_errno(WLR_ERROR, "fcntl(F_DUPFD_CLOEXEC) failed"); wlr_log_errno(WLR_ERROR, "fcntl(F_DUPFD_CLOEXEC) failed");
return false; goto error;
} }
} }
return true; return true;
error:
for (int j = 0; j < i; j++) {
close(dst->fd[i]);
dst->fd[j] = -1;
}
dst->n_planes = 0;
return false;
} }