Fix fuzzer bug b/309756937 (#35279)

Built on #35278, which should be landed first

Always fail parsing when `grpclb_client_stats` is included in headers -- it's a meaningless value and the only reason to include it would be some sort of attack.

Closes #35279

COPYBARA_INTEGRATE_REVIEW=https://github.com/grpc/grpc/pull/35279 from ctiller:fuzz-309756937 545448c4de
PiperOrigin-RevId: 590745978
test_590826993
Craig Tiller 12 months ago
parent 33a4c8c994
commit d00582bd82
  1. 3
      src/core/lib/transport/metadata_batch.h
  2. 2
      src/objective-c/tests/Hosts/ios-host/AppDelegate.github.h
  3. 2
      src/objective-c/tests/Hosts/ios-host/AppDelegate.github.m
  4. 2
      src/objective-c/tests/Hosts/ios-host/Info.github.plist
  5. 2
      src/objective-c/tests/Hosts/ios-host/main.github.m
  6. BIN
      test/core/transport/chaotic_good/frame_fuzzer_corpus/5691448031772672

@ -410,7 +410,8 @@ struct GrpcLbClientStatsMetadata {
static const char* DisplayMemento(MementoType) {
return "<internal-lb-stats>";
}
static MementoType ParseMemento(Slice, bool, MetadataParseErrorFn) {
static MementoType ParseMemento(Slice, bool, MetadataParseErrorFn error) {
error("not a valid value for grpclb_client_stats", Slice());
return nullptr;
}
};

@ -22,4 +22,4 @@
@property(strong, nonatomic) UIWindow* window;
@end
@end

@ -24,4 +24,4 @@
@implementation AppDelegate
@end
@end

@ -38,4 +38,4 @@
<string>UIInterfaceOrientationLandscapeRight</string>
</array>
</dict>
</plist>
</plist>

@ -23,4 +23,4 @@ int main(int argc, char* argv[]) {
@autoreleasepool {
return UIApplicationMain(argc, argv, nil, NSStringFromClass([AppDelegate class]));
}
}
}

Loading…
Cancel
Save