fix(upb): escape field key names in jsonenc_fieldval when encoding (#31010)

### Summary

`jsonenc_fieldval` in `upb/json/encode.c` writes an object key with `jsonenc_printf(e, "\"%s\":", name)`, where `name` comes from `upb_FieldDef_JsonName` (or `upb_FieldDef_Name` with `upb_JsonEncode_UseProtoNames`). Field names are checked as identifiers when a descriptor is added to a def pool, but `json_name` is copied from the `FieldDescriptorProto` unchanged. A `json_name` that contains a quote, a backslash or a control character therefore reaches the output unescaped. With the tests added here, a field whose `json_name` is

```
a":1,"b
```

is encoded as `{"a":1,"b":1}`, which is a two-key object, and a `json_name` containing a newline, a backslash or a tab produces invalid JSON. Values, map keys, struct keys and enum names in the same encoder already go through `jsonenc_string` or `jsonenc_stringbody`.

### Fix

In `jsonenc_fieldval`, emit the key with `jsonenc_string`, followed by `:`. The extension branch, which formats the full name in brackets, now passes the name through `jsonenc_stringbody`. Extension full names are identifier-checked by the def builder, so the extension change has no observable effect for descriptors the def pool accepts; it only makes the two branches consistent. It can be dropped if you prefer the smaller change.

This change does not touch `jsonenc_fieldpath`.

### Relationship to #28678

#28678 makes the same `json_name` change in `jsonenc_fieldval`. It also adds a character allow-list to `jsonenc_fieldpath`. Applied to current main, that allow-list makes the existing `EncodeFieldMaskEscapedPaths` fail, because that test expects paths containing a newline, a tab and a quote to encode and be escaped. This change leaves `jsonenc_fieldpath` alone and `EncodeFieldMaskEscapedPaths` keeps passing. If #28678 lands first, the `jsonenc_fieldval` part of this change is redundant.

### Testing

Added to `upb/json/encode_test.cc`, with a new `JsonNameEscape` message in `upb/json/test.proto`:

- `EncodeJsonNamePositiveControl` (also with `upb_JsonEncode_UseProtoNames`)
- `EncodeJsonNameWithQuotesAndBackslashesIsEscaped` (also decodes the output with `upb_JsonDecode` and checks the field round-trips)
- `EncodeJsonNameWithControlCharactersIsEscaped`

```
bazel test //upb/json:encode_test
```

passes with the change (22 tests, including `EncodeFieldMaskEscapedPaths`). Without the change, the two escaping tests fail and the others pass. Built with clang 22.1.8 and Bazel 8.7.0 on x86_64 Linux, also under ASan and UBSan.

AI assistance was used in drafting and validating this change; the patch and tests were reviewed and verified by the author.

Closes #31010

COPYBARA_INTEGRATE_REVIEW=https://github.com/protocolbuffers/protobuf/pull/31010 from professor-moody:fix-upb-json-encode-field-key-escaping ff14f061c3b397112c241067e99e1e80aa265c28
PiperOrigin-RevId: 1015144727
3 files changed
tree: dfaa86f35761590290d0a7033f4f088bd7df1c83
  1. .bazelci/
  2. .bcr/
  3. .github/
  4. bazel/
  5. benchmarks/
  6. build_defs/
  7. ci/
  8. cmake/
  9. compatibility/
  10. conformance/
  11. csharp/
  12. docs/
  13. editions/
  14. editors/
  15. examples/
  16. go/
  17. hpb/
  18. hpb_generator/
  19. java/
  20. lua/
  21. objectivec/
  22. patches/
  23. php/
  24. pkg/
  25. python/
  26. ruby/
  27. rust/
  28. src/
  29. third_party/
  30. toolchain/
  31. upb/
  32. upb_generator/
  33. .bazelignore
  34. .bazeliskrc
  35. .bazelrc
  36. .clang-format
  37. .gitattributes
  38. .gitignore
  39. .gitmodules
  40. .readthedocs.yml
  41. appveyor.bat
  42. appveyor.yml
  43. bazel9.bazelrc
  44. BUILD.bazel
  45. CMakeLists.txt
  46. CODE_OF_CONDUCT.md
  47. CONTRIBUTING.md
  48. CONTRIBUTORS.txt
  49. Disable_bundle_install.patch
  50. fix_permissions.sh
  51. generate_descriptor_proto.sh
  52. global.json
  53. LICENSE
  54. maven_dev_install.json
  55. maven_install.json
  56. MODULE.bazel
  57. PrivacyInfo.xcprivacy
  58. protobuf.bzl
  59. Protobuf.podspec
  60. protobuf_deps.bzl
  61. protobuf_release.bzl
  62. protobuf_version.bzl
  63. README.md
  64. regenerate_stale_files.sh
  65. SECURITY.md
  66. version.json
  67. WORKSPACE
  68. WORKSPACE.bzlmod
README.md

Protocol Buffers - Google's data interchange format

OpenSSF Scorecard

Copyright 2008 Google LLC

Overview

Protocol Buffers (a.k.a., protobuf) are Google's language-neutral, platform-neutral, extensible mechanism for serializing structured data. You can learn more about it in protobuf's documentation.

This README file contains protobuf installation instructions. To install protobuf, you need to install the protocol compiler (used to compile .proto files) and the protobuf runtime for your chosen programming language.

Working With Protobuf Source Code

Most users will find working from supported releases to be the easiest path.

If you choose to work from the head revision of the main branch your build will occasionally be broken by source-incompatible changes and insufficiently-tested (and therefore broken) behavior.

If you are using C++ or otherwise need to build protobuf from source as a part of your project, you should pin to a release commit on a release branch.

This is because even release branches can experience some instability in between release commits.

Bazel with Bzlmod

Protobuf supports Bzlmod with Bazel 8 +. Users should specify a dependency on protobuf in their MODULE.bazel file as follows.

bazel_dep(name = "protobuf", version = <VERSION>)

Users can optionally override the repo name, such as for compatibility with WORKSPACE.

bazel_dep(name = "protobuf", version = <VERSION>, repo_name = "com_google_protobuf")

Bazel with WORKSPACE

Users can also add the following to their legacy WORKSPACE file.

Note that with the release of 30.x there are a few more load statements to properly set up rules_java and rules_python.

http_archive(
    name = "com_google_protobuf",
    strip_prefix = "protobuf-VERSION",
    sha256 = ...,
    url = ...,
)

load("@com_google_protobuf//:protobuf_deps.bzl", "protobuf_deps")

protobuf_deps()

load("@rules_java//java:rules_java_deps.bzl", "rules_java_dependencies")

rules_java_dependencies()

load("@rules_java//java:repositories.bzl", "rules_java_toolchains")

rules_java_toolchains()

load("@rules_python//python:repositories.bzl", "py_repositories")

py_repositories()

Protobuf Compiler Installation

The protobuf compiler is written in C++. If you are using C++, please follow the C++ Installation Instructions to install protoc along with the C++ runtime.

For non-C++ users, the simplest way to install the protocol compiler is to download a pre-built binary from our GitHub release page.

In the downloads section of each release, you can find pre-built binaries in zip packages: protoc-$VERSION-$PLATFORM.zip. It contains the protoc binary as well as a set of standard .proto files distributed along with protobuf.

If you are looking for an old version that is not available in the release page, check out the Maven repository.

These pre-built binaries are only provided for released versions. If you want to use the github main version at HEAD, or you need to modify protobuf code, or you are using C++, it's recommended to build your own protoc binary from source.

If you would like to build protoc binary from source, see the C++ Installation Instructions.

Protobuf Runtime Installation

Protobuf supports several different programming languages. For each programming language, you can find instructions in the corresponding source directory about how to install protobuf runtime for that specific language:

LanguageSource
C++ (include C++ runtime and protoc)src
Javajava
Pythonpython
Objective-Cobjectivec
C#csharp
Rubyruby
Goprotocolbuffers/protobuf-go
PHPphp
Dartdart-lang/protobuf
JavaScriptprotocolbuffers/protobuf-javascript

Quick Start

The best way to learn how to use protobuf is to follow the tutorials in our developer guide.

If you want to learn from code examples, take a look at the examples in the examples directory.

Documentation

The complete documentation is available at the Protocol Buffers doc site.

Support Policy

Read about our version support policy to stay current on support timeframes for the language libraries.

Developer Community

To be alerted to upcoming changes in Protocol Buffers and connect with protobuf developers and users, join the Google Group.