Marcus Meissner : msi: Check return value of msi_string2idW (Coverity).

Alexandre Julliard julliard at winehq.org
Wed Aug 31 13:19:22 CDT 2011


Module: wine
Branch: master
Commit: ee12b4feed8885a4c22716eade4154101ec5577f
URL:    http://source.winehq.org/git/wine.git/?a=commit;h=ee12b4feed8885a4c22716eade4154101ec5577f

Author: Marcus Meissner <meissner at suse.de>
Date:   Tue Aug 30 18:17:21 2011 +0200

msi: Check return value of msi_string2idW (Coverity).

---

 dlls/msi/streams.c |    6 ++++--
 1 files changed, 4 insertions(+), 2 deletions(-)

diff --git a/dlls/msi/streams.c b/dlls/msi/streams.c
index 56b38e3..71616e9 100644
--- a/dlls/msi/streams.c
+++ b/dlls/msi/streams.c
@@ -296,10 +296,12 @@ static UINT STREAMS_get_column_info( struct tagMSIVIEW *view, UINT n, LPCWSTR *n
 static UINT streams_find_row(MSISTREAMSVIEW *sv, MSIRECORD *rec, UINT *row)
 {
     LPCWSTR str;
-    UINT i, id, data;
+    UINT r, i, id, data;
 
     str = MSI_RecordGetString(rec, 1);
-    msi_string2idW(sv->db->strings, str, &id);
+    r = msi_string2idW(sv->db->strings, str, &id);
+    if (r != ERROR_SUCCESS)
+        return r;
 
     for (i = 0; i < sv->num_rows; i++)
     {




More information about the wine-cvs mailing list