Bug #23181 wrong order of memset() arguments
Submitted: 11 Oct 2006 15:49 Modified: 16 Oct 2006 14:56
Reporter: Arkadiusz Miskiewicz (Basic Quality Contributor) Email Updates:
Status: Closed Impact on me:
None 
Category:MySQL Administrator Severity:S3 (Non-critical)
Version: OS:
Assigned to: Mike Lischke CPU Architecture:Any

[11 Oct 2006 15:49] Arkadiusz Miskiewicz
Description:
http://svn.mysql.com/fisheye/browse/~raw,r=491/mysql-administrator/trunk/testing/unit-test...

contains:
memset(&status, sizeof(status), 0);

which has wrong order of arguments.

I guess that applies to other branches, too.

How to repeat:
less that file

Suggested fix:
memset(&status, 0, sizeof(status));
[11 Oct 2006 18:32] Sveta Smirnova
Thank you for the report.

Verified as described.
[16 Oct 2006 14:56] Mike Lischke
Thank you for your bug report. This issue has been committed to our source repository of that product and will be incorporated into the next release.

If necessary, you can access the source repository and build the latest available version, including the bug fix. More information about accessing the source trees is available at

    http://dev.mysql.com/doc/en/installing-source.html

PS: Good catch! I checked all other files in our repositories. All of them are fine where memset is used.
[16 Oct 2006 15:33] Mike Lischke
Actually, I did not search all files in the first run. I found 5-6 other places where this was wrong. Thanks again for the finding and report.