[Bug 11479] New: db: \r should be stripped from identifiers in SQL

http://bugs.winehq.org/show_bug.cgi?id=11479

Summary: db: \r should be stripped from identifiers in SQL
           Product: Wine
           Version: CVS/GIT
          Platform: Other
        OS/Version: other
            Status: UNCONFIRMED
          Severity: enhancement
          Priority: P2
         Component: msi
        AssignedTo: [e-mail]
        ReportedBy: [e-mail]

Created an attachment (id=10620)
 --> (http://bugs.winehq.org/attachment.cgi?id=10620)
stupid patch and testcase

While trying to compile the basic example in Wix, an query string is sent in to
MsiDatabaseOpenView() for which MSI_ParseSQL return 1. Eg:

trace:msi:MSI_DatabaseOpenViewW L"CREATE TABLE `_SummaryInformation` (
`PropertyId` INT NOT NULL, `Value\r` CHAR(255\r) NOT NULL LOCALIZABLE PRIMARY
KEY `PropertyId\r`)" 0x61ed98
trace:msi:MSI_ParseSQL Parse returned 1

To reproduce this error, the instructions in bug 11414, comment 10 applies.
Attached is a new test to msi/tests/db.c stupid and naive patch which just
strips the trailing \r if present.