Bug #34967 find error in MySqlString.cs

• Submitted: 1 Mar 2008 6:56
• Modified: 3 Apr 2008 19:49
• Reporter: joshua zhu
• Status: No Feedback
• Category: Connector / NET
• Severity: S3 (Non-critical)
• Version: 5.1.5
• OS: Windows
• CPU Architecture: Any
• Tags: EscapeString

[1 Mar 2008 6:56] joshua zhu

Description:
private string EscapeString(string s)
{
	s = s.Replace("\\", "\\\\");
	s = s.Replace("\'", "\\\'");
	s = s.Replace("\"", "\\\"");
	s = s.Replace("`", "\\`");
	s = s.Replace("?, "\\?);	// error
	s = s.Replace("?, "\\?);	// error
	s = s.Replace("?, "\\?);	// error
	return s;
}

How to repeat:
private string EscapeString(string s)
{
	s = s.Replace("\\", "\\\\");
	s = s.Replace("\'", "\\\'");
	s = s.Replace("\"", "\\\"");
	s = s.Replace("`", "\\`");
	s = s.Replace("?, "\\?);	// error
	s = s.Replace("?, "\\?);	// error
	s = s.Replace("?, "\\?);	// error
	return s;
}

Suggested fix:
 :(

[3 Mar 2008 19:49] Tonci Grgin

Hi Joshua and thanks for your report. Please elaborate a bit and attach some test case showing what's bothering you.

[3 Apr 2008 23:00] Bugs System

No feedback was provided for this bug for over a month, so it is
being suspended automatically. If you are able to provide the
information that was originally requested, please do so and change
the status of the bug back to "Open".