diff options
author | Yoonsoo Kim <yoonsoo.kim@mongodb.com> | 2021-08-10 17:39:49 +0000 |
---|---|---|
committer | Evergreen Agent <no-reply@evergreen.mongodb.com> | 2021-08-10 18:34:30 +0000 |
commit | 8016ab594857e6e53daff9ae6bba7f9f2336ad1e (patch) | |
tree | 2946d3a3475da763e3c6b13ca80362333b7756de /src/mongo/rpc/protocol_fuzzer.cpp | |
parent | 3888d215464a081c831074918ad590a8e3ba74a1 (diff) | |
download | mongo-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.cpp | 15 |
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())); |