Skip to content

GH-49760: [Doc][C++] Elaborate on what constitutes a vulnerability - #55271

Merged
pitrou merged 1 commit into
apache:mainfrom
pitrou:gh49760-cpp-security-vuln
Oct 27, 2026
Merged

pitrou merged 1 commit into
apache:mainfrom
pitrou:gh49760-cpp-security-vuln

Conversation

@pitrou

@pitrou pitrou commented Oct 26, 2026 •

Copy link
Copy Markdown
Member

Rationale for this change

We often get private security reports for some issues which don't threaten confidentiality or integrity.

For these kinds of issues, we would rather have them reported through the usual public means, as it follows our open development principles more closely, and lessens the required paperwork (no CVE, etc.).

What changes are included in this PR?

Add a paragraph in the C++ security model about what should be reported as a security vulnerability vs. a regular bug.

Are these changes tested?

N/A.

Are there any user-facing changes?

No.

Was AI used for this PR?

In accordance to the AI generation guidelines, please disclose below whether and how AI was used in this PR.

PR code and description written by:

  • Human
  • AI

Reviewed before submission by:

  • Human
  • AI
  • Not reviewed

Security model for Arrow C++

Arrow Rust `Safety and Security <https://docs.rs/arrow/latest/arrow/#security>`__
Security model for Arrow Rust

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@alamb @tustvold @paleolimbot Could you look at these non-C++ specific changes?

@pitrou

pitrou commented Oct 26, 2026

Copy link
Copy Markdown
Member Author

@zanmato1984 @HuaHuaY @taepper @westonpace @emkornfield Would you like to take a look?

@github-actions github-actions Bot added the awaiting review Awaiting review label Oct 26, 2026
@pitrou

pitrou commented Oct 26, 2026

Copy link
Copy Markdown
Member Author

@github-actions crossbow submit preview-docs

@github-actions github-actions Bot added awaiting committer review Awaiting committer review and removed awaiting review Awaiting review labels Oct 26, 2026
@github-actions

Copy link
Copy Markdown

Revision: 32291c5

Submitted crossbow builds: ursacomputing/crossbow @ actions-a1be7c45db

Task Status
preview-docs GitHub Actions

@paleolimbot paleolimbot left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

This seems like a reasonable change...the C++ bits apply to nanoarrow too but we haven't had many security reports until the last few weeks.

Most reports claim that crashes have a privacy component (e.g., use after free), but hopefully this helps. Having these as open issues makes it much simpler to get them fixed (e.g., doesn't have to be somebody specifically watching the private list).

@github-actions github-actions Bot added awaiting merge Awaiting merge and removed awaiting committer review Awaiting committer review labels Oct 26, 2026
@pitrou
pitrou merged commit 96207d7 into apache:main Oct 27, 2026
37 of 39 checks passed
@pitrou pitrou removed the awaiting merge Awaiting merge label Oct 27, 2026
@pitrou
pitrou deleted the gh49760-cpp-security-vuln branch October 27, 2026 08:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants