Inconsistent user of character string lengths
Bug #1370749 reported by
Roberta Marton
This bug affects 1 person
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
Trafodion |
Fix Released
|
Low
|
Cliff Gray |
Bug Description
In Context.cpp (2014-09-16) code base:
Line 4853: char authNameFromTab
Looks like authQuery will make the assumption that 600 (actually 599) is long enough. Shouldn't this at least be a defined constant rather than hard-coded?
Line 5097: char usersNameFromUs
600 seems to be a popular number. Should be a defined enum or const int in a header file somewhere.
We do already have a literal that defines the length of authorization names. This should be used instead of 600.
Changed in trafodion: | |
importance: | Undecided → Low |
assignee: | nobody → Cliff Gray (cliff-gray) |
tags: | added: sql-security |
Changed in trafodion: | |
milestone: | none → r1.1 |
Changed in trafodion: | |
status: | Fix Committed → Fix Released |
To post a comment you must log in.
Fix complete, waiting until r1.0 is released before delivering.