ubuf is too small to hold a human readable 64 bit value
ClosedPublic
Actions

Authored by bret_ketchum_dell.com on Oct 21 2020, 4:57 PM.

Tags

None

Referenced Files

	F110484703: D26893.diff
	Wed, Feb 19, 2:14 AM

	Unknown Object (File)
	Sat, Jan 25, 9:55 PM

	Unknown Object (File)
	Sat, Jan 25, 9:03 PM

	Unknown Object (File)
	Jan 17 2025, 7:27 PM

	Unknown Object (File)
	Jan 14 2025, 10:37 AM

	Unknown Object (File)
	Jan 14 2025, 10:37 AM

	Unknown Object (File)
	Jan 14 2025, 10:37 AM

	Unknown Object (File)
	Jan 14 2025, 9:49 AM

View All 59 Files

Subscribers

imp

Details

Reviewers

markj
mav
vangyzen

Commits

rS366914: ntb_tool: ubuf is too small to hold a human readable 64 bit value

Summary

ubuf buffer is too small. It should be 18 if a NULL is not needed, or 19 to hold the NULL terminator for the full 64-BIT value plus the 0x prefix.

Test Plan

Run ntb_tool.

Diff Detail

Lint

Lint Skipped

Unit

Tests Skipped

Event Timeline

bret_ketchum_dell.com requested review of this revision.Oct 21 2020, 4:57 PM

bret_ketchum_dell.com created this revision.

markj accepted this revision.Oct 21 2020, 5:00 PM

This revision is now accepted and ready to land.Oct 21 2020, 5:00 PM

mav accepted this revision.Oct 21 2020, 5:06 PM

vangyzen accepted this revision.Oct 21 2020, 5:07 PM

Closed by commit rS366914: ntb_tool: ubuf is too small to hold a human readable 64 bit value (authored by vangyzen). · Explain WhyOct 21 2020, 5:12 PM

This revision was automatically updated to reflect the committed changes.

vangyzen added a commit: rS366914: ntb_tool: ubuf is too small to hold a human readable 64 bit value.

Herald added a subscriber: imp. · View Herald TranscriptOct 21 2020, 5:12 PM

Revision Contents
Changeset List

Path

Size

sys/

dev/

ntb/

test/

ntb_tool.c

2 lines

Diff 78556

View Options

sys/dev/ntb/test/ntb_tool.c

ubuf is too small to hold a human readable 64 bit valueClosedPublicActions