Skip to content

[ruby] Return nil for c functions expected to return a VALUE#39214

Closed
rwstauner wants to merge 1 commit intogrpc:masterfrom
Shopify:rwstauner/return-values
Closed

[ruby] Return nil for c functions expected to return a VALUE#39214
rwstauner wants to merge 1 commit intogrpc:masterfrom
Shopify:rwstauner/return-values

Conversation

@rwstauner
Copy link
Copy Markdown
Contributor

Otherwise any attempt to access the return value will segfault.

Copy link
Copy Markdown
Contributor

@apolcyn apolcyn left a comment

Choose a reason for hiding this comment

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

thanks!

@apolcyn apolcyn added the release notes: yes Indicates if PR needs to be in release notes label May 5, 2025
Otherwise any attempt to access the return value will segfault.
@rwstauner rwstauner force-pushed the rwstauner/return-values branch from 7373e82 to 98bd004 Compare June 9, 2025 18:26
@rwstauner rwstauner requested a review from yashykt June 9, 2025 18:27
@copybara-service copybara-service bot closed this in 9e88cab Jun 9, 2025
anniefrchz pushed a commit to anniefrchz/grpc that referenced this pull request Jun 25, 2025
)

Otherwise any attempt to access the return value will segfault.

Closes grpc#39214

COPYBARA_INTEGRATE_REVIEW=grpc#39214 from Shopify:rwstauner/return-values 7373e82
PiperOrigin-RevId: 769307122
paulosjca pushed a commit to paulosjca/grpc that referenced this pull request Aug 23, 2025
)

Otherwise any attempt to access the return value will segfault.

Closes grpc#39214

COPYBARA_INTEGRATE_REVIEW=grpc#39214 from Shopify:rwstauner/return-values 7373e82
PiperOrigin-RevId: 769307122
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

lang/ruby release notes: yes Indicates if PR needs to be in release notes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants