diff options
author | Rich Megginson <rmeggins@redhat.com> | 2008-08-28 16:54:43 +0000 |
---|---|---|
committer | Rich Megginson <rmeggins@redhat.com> | 2008-08-28 16:54:43 +0000 |
commit | 4a86db73ad8ee0fa37307415616df050ac0a2f7c (patch) | |
tree | a8ecc3be64d29b05dbf5b657fc8ed615d3d842cb /modules.awk | |
parent | c6b81ccab7205fac0858af897c9931223d518516 (diff) | |
download | ds-Directory71RtmBranch.tar.gz ds-Directory71RtmBranch.tar.xz ds-Directory71RtmBranch.zip |
Resolves: bug 413531ds71sp7rc_20080812Directory71Sp7RtmRhelHpuxSolaris_20080812Directory71Sp7RtmRhelHpuxSolarisDirectory71RtmBranch
Bug Description: Web browser accepted languages configuration causes dsgw CGI binaries to segfault
Reviewed by: nhosoi (Thanks!)
Files: ldapserver/lib/libsi18n/acclanglist.c adminserver/lib/libsi18n/acclanglist.c
Branch: Directory71RtmBranch
Fix Description: The CGI code looks for localized files using the following order: /path/lang_co, /path/lang, /path/"en". It uses the HTTP Accept-Language string to look for languages the user wants, in the order that the user wants. The user may specify "fr_CA, fr_FR, fr_BE" for example - the user does not have to specify "fr". But we want to use /path/"fr" in that case, if we don't have /path/"fr_CA" nor any of the other lang_co directories. The code in XP_AccLangList attempts to scan through the parsed list and add the two char lang codes if they are not already in the list. However, the code that appended them did not check for buffer overflow The new code, while not being quite as efficient as the old code, does correctly check for overflow and should be easier to read.
Platforms tested: RHEL5, HP-UX
Flag Day: no
Doc impact: no
QA impact: Yes
New Tests integrated into TET: Yes - we should add something to the Admin Server and DSGW test plans to test this case.
Diffstat (limited to 'modules.awk')
0 files changed, 0 insertions, 0 deletions