summaryrefslogtreecommitdiff
path: root/src/mongo/rpc/protocol_fuzzer.cpp
diff options
context:
space:
mode:
authorYoonsoo Kim <yoonsoo.kim@mongodb.com>2021-08-10 17:39:49 +0000
committerEvergreen Agent <no-reply@evergreen.mongodb.com>2021-08-10 18:34:30 +0000
commit8016ab594857e6e53daff9ae6bba7f9f2336ad1e (patch)
tree2946d3a3475da763e3c6b13ca80362333b7756de /src/mongo/rpc/protocol_fuzzer.cpp
parent3888d215464a081c831074918ad590a8e3ba74a1 (diff)
downloadmongo-8016ab594857e6e53daff9ae6bba7f9f2336ad1e.tar.gz
SERVER-58140 Remove legacy update/delete parsers
Diffstat (limited to 'src/mongo/rpc/protocol_fuzzer.cpp')
-rw-r--r--src/mongo/rpc/protocol_fuzzer.cpp15
1 files changed, 5 insertions, 10 deletions
diff --git a/src/mongo/rpc/protocol_fuzzer.cpp b/src/mongo/rpc/protocol_fuzzer.cpp
index 19876d0ef47..a9edfba382b 100644
--- a/src/mongo/rpc/protocol_fuzzer.cpp
+++ b/src/mongo/rpc/protocol_fuzzer.cpp
@@ -75,11 +75,10 @@ void doFuzzing(ConstDataRangeCursor fuzzedData) try {
msg = uassertStatusOK(compression.manager.decompressMessage(msg));
}
- DbMessage dbMsgObj(msg);
switch (msg.operation()) {
case dbMsg: {
- auto request = rpc::opMsgRequestFromAnyProtocol(msg);
+ auto request = OpMsgRequest::parseOwned(msg);
validateBSON(request.body);
for (const auto& docSeq : request.sequences) {
for (const auto& doc : docSeq.objs) {
@@ -87,22 +86,18 @@ void doFuzzing(ConstDataRangeCursor fuzzedData) try {
}
}
} break;
- case dbUpdate: {
- auto op = UpdateOp::parseLegacy(msg);
- } break;
case dbInsert: {
auto op = InsertOp::parseLegacy(msg);
} break;
- case dbDelete: {
- auto op = DeleteOp::parseLegacy(msg);
- }
case dbQuery: {
+ DbMessage dbMsgObj(msg);
QueryMessage q(dbMsgObj);
} break;
- // TODO these message types don't have discrete parsers. We should move their parsing out
- // of the ServiceEntryPointCommon so we can actually
+ // These message types don't have parsers because they are no longer supported.
case dbGetMore:
case dbKillCursors:
+ case dbDelete:
+ case dbUpdate:
break;
default:
invariant(!isSupportedRequestNetworkOp(msg.operation()));