qiurui
/
qtgrpc
mirror of https://github.com/qt/qtgrpc.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

Assert if QGrpcOperationContext::serializer returns null 

QGrpcOperationContext shall be constructed with the valid serializer.
Let's avoid unnecessary error handling and assert if
QGrpcOperationContext doesn't contain a valid pointer to a serializer.

[ChangeLog][Grpc] QAbstractGrpcChannel implementations now are obliged
to supply QGrpcContext with the valid serializer. Any attempt to
deserialize the returned message with null serializer now will lead to
assert or SEGV.

Pick-to: 6.8
Change-Id: I1f9142244bf5b5ba7a3d04965649d0a0fbb143b9
Reviewed-by: Dennis Oberst <dennis.oberst@qt.io>
This commit is contained in:
Alexey Edelev 2024-08-24 14:19:06 +02:00
parent a82ac155d7
commit 271764cddf
1 changed files with 4 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

9
src/grpc/qgrpcoperation.cpp
View File

@ -116,7 +116,8 @@ bool QGrpcOperation::read(QProtobufMessage *message) const
"Can't read to nullptr QProtobufMessage");
Q_D(const QGrpcOperation);
const auto ser = d->operationContext->serializer();
return ser && ser->deserialize(message, d->data);
Q_ASSERT_X(ser, "QGrpcOperation", "The serializer is null");
return ser->deserialize(message, d->data);
}
/*!
@ -144,8 +145,7 @@ QAbstractProtobufSerializer::DeserializationError QGrpcOperation::deserializatio
{
Q_D(const QGrpcOperation);
const auto ser = d->operationContext->serializer();
if (!ser)
return QAbstractProtobufSerializer::NoDeserializerError;
Q_ASSERT_X(ser, "QGrpcOperation", "The serializer is null");
return ser->deserializationError();
}
@ -160,8 +160,7 @@ QString QGrpcOperation::deserializationErrorString() const
{
Q_D(const QGrpcOperation);
const auto ser = d->operationContext->serializer();
if (!ser)
return QStringLiteral("serializer not available");
Q_ASSERT_X(ser, "QGrpcOperation", "The serializer is null");
return ser->deserializationErrorString();
}