3 files changed, 70 insertions, 3 deletions
diff --git a/src/gui/painting/qpen.cpp b/src/gui/painting/qpen.cpp
index c0b3769c2d..b661057f64 100644
--- a/src/gui/painting/qpen.cpp
+++ b/src/gui/painting/qpen.cpp
@@ -327,17 +327,29 @@ QPen::QPen(const QBrush &brush, qreal width, Qt::PenStyle s, Qt::PenCapStyle c,
 QPen::QPen(const QPen &p)
 {
     d = p.d;
-    d->ref.ref();
+    if (d)
+        d->ref.ref();
 }
 
 
 /*!
+    \fn QPen::QPen(QPen &&pen)
+    \since 5.4
+
+    Constructs a pen that is moved from the given \a pen.
+
+    The moved-from pen can only be assigned to, copied, or
+    destroyed. Any other operation (prior to assignment) leads to
+    undefined behavior.
+*/
+
+/*!
     Destroys the pen.
 */
 
 QPen::~QPen()
 {
-    if (!d->ref.deref())
+    if (d && !d->ref.deref())
         delete d;
 }
 
@@ -373,7 +385,7 @@ void QPen::detach()
 
 QPen &QPen::operator=(const QPen &p)
 {
-    qAtomicAssign(d, p.d);
+    QPen(p).swap(*this);
     return *this;
 }
 
diff --git a/src/gui/painting/qpen.h b/src/gui/painting/qpen.h
index c5144f784f..486f699476 100644
--- a/src/gui/painting/qpen.h
+++ b/src/gui/painting/qpen.h
@@ -72,6 +72,8 @@ public:
 
     QPen &operator=(const QPen &pen);
 #ifdef Q_COMPILER_RVALUE_REFS
+    inline QPen(QPen &&other)
+        : d(other.d) { other.d = 0; }
     inline QPen &operator=(QPen &&other)
     { qSwap(d, other.d); return *this; }
 #endif
diff --git a/tests/auto/gui/painting/qpen/tst_qpen.cpp b/tests/auto/gui/painting/qpen/tst_qpen.cpp
index 07c996d026..a373c51710 100644
--- a/tests/auto/gui/painting/qpen/tst_qpen.cpp
+++ b/tests/auto/gui/painting/qpen/tst_qpen.cpp
@@ -57,6 +57,8 @@ public:
 private slots:
     void getSetCheck();
     void swap();
+    void move();
+    void move_assign();
     void operator_eq_eq();
     void operator_eq_eq_data();
 
@@ -101,6 +103,57 @@ void tst_QPen::swap()
     QCOMPARE(p2.color(), QColor(Qt::black));
 }
 
+void tst_QPen::move()
+{
+    QPen p1(Qt::black);
+
+    // check that moving does the right thing:
+    QPen p2 = qMove(p1); // could be move or copy construction, so don't check p1's state
+    QCOMPARE(p2.color(), QColor(Qt::black));
+
+    // this, executed ehre, would crash:
+    // QVERIFY(p1.style() != Qt::NoPen);
+
+    // check that moved-from QPen p1 can still be safely copied:
+    const QPen p3 = p1;
+
+    // check that moved-from QPen p1 can still be safely assigned to:
+    const QPen p4(Qt::yellow);
+    p1 = p4;
+    QCOMPARE(p1.color(), QColor(Qt::yellow));
+
+    // check that moved-from QPens p2, p3 can still be safely destroyed:
+    QPen p5 = qMove(p2);
+
+    // intentionally no more statements beyond this point
+}
+
+void tst_QPen::move_assign()
+{
+    QPen p1(Qt::black), p2(Qt::white);
+
+    // check that moving does the right thing:
+    p2 = qMove(p1); // could be move or copy assignment, so don't check p1's state
+    QCOMPARE(p2.color(), QColor(Qt::black));
+
+    // check that move-assigned-from QPen p1 can still be used, albeit
+    // with undocumented state (it's p2's original state):
+    QVERIFY(p1.style() != Qt::NoPen);
+
+    // check that moved-from QPen p1 can still be safely copied:
+    const QPen p3 = p1;
+
+    // check that moved-from QPen p1 can still be safely assigned to:
+    const QPen p4(Qt::yellow);
+    p1 = p4;
+    QCOMPARE(p1.color(), QColor(Qt::yellow));
+
+    // check that moved-from QPens p2, p3 can still be safely destroyed:
+    QPen p5;
+    p5 = qMove(p2);
+
+    // intentionally no more statements beyond this point
+}
 
 tst_QPen::tst_QPen()