[Bug 44293] New: Possible access to unintended field in "wine/tools/widl/ header.c" line 1292

https://bugs.winehq.org/show_bug.cgi?id=44293

Bug ID: 44293
           Summary: Possible access to unintended field in
                    "wine/tools/widl/header.c" line 1292
           Product: Wine
           Version: unspecified
          Hardware: x86
                OS: Linux
            Status: UNCONFIRMED
          Severity: normal
          Priority: P2
         Component: tools
          Assignee: [e-mail]
          Reporter: [e-mail]
      Distribution: ---

While experimenting with a CodeSonar plugin we develop, we noticed a
potential bug in file "wine/tools/widl/header.c" line 1292, function
write_locals

write_type_decl_left(fp, type_function_get_rettype(func->type));
fprintf(fp, " __RPC_STUB %s_%s_Stub(\n", iface->name, get_name(m)); //HERE
write_args(fp, type_get_function_args(func->type), iface->name, 1, TRUE);
fprintf(fp, ")");

Shouldn't func be used instead of m in the mentioned line? The lines around
that line both make use of func and it is not clear why m is in use in line
1292.

Thanks,
Petru Florin Mihancea

[Bug 44293] Possible access to unintended field in "wine/tools/widl/ header.c" line 1292

https://bugs.winehq.org/show_bug.cgi?id=44293

Alexandre Julliard  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|UNCONFIRMED                 |RESOLVED
         Resolution|---                         |INVALID

--- Comment #1 from Alexandre Julliard  ---
It's more logical to have proxy and stub use the same name.

[Bug 44293] Possible access to unintended field in "wine/tools/widl/ header.c" line 1292

https://bugs.winehq.org/show_bug.cgi?id=44293

André H.  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |[e-mail]
             Status|RESOLVED                    |CLOSED

--- Comment #2 from André H.  ---
closing invalid