QRenderTargetSelector fixes:

- Property copy drawBuffers when cloning
- Check that the target isn't null when we try to set its parent

Change-Id: I61b1f8972f9813efa371b929b26490d0a0b577c2
Reviewed-by: Andy Nichols <andy.nichols@theqtcompany.com>

1 files changed, 2 insertions, 1 deletions

diff --git a/src/render/framegraph/qrendertargetselector.cpp b/src/render/framegraph/qrendertargetselector.cpp
index 58529ae09..c3fca07a7 100644
--- a/src/render/framegraph/qrendertargetselector.cpp
+++ b/src/render/framegraph/qrendertargetselector.cpp
@@ -62,6 +62,7 @@ void QRenderTargetSelector::copy(const QNode *ref)
 
     const QRenderTargetSelector *other = static_cast<const QRenderTargetSelector*>(ref);
 
+    setDrawBuffers(other->drawBuffers());
     if (other->d_func()->m_target)
         setTarget(qobject_cast<QRenderTarget *>(QNode::clone(other->d_func()->m_target)));
 }
@@ -83,7 +84,7 @@ void QRenderTargetSelector::setTarget(QRenderTarget *target)
         d->m_target = target;
 
         // For inline declaration cases
-        if (!target->parent())
+        if (target != Q_NULLPTR && !target->parent())
             target->setParent(this);
         emit targetChanged();
     }