Why ar you creating a new action in the drop event instead of reusing the original one? Are you sure beforeAction points to a valid action (can you retrieve its name) after using actionAt? To me it would seem more appropriate to use the position from the drop event and not from QCursor::pos().