fusion: Explicitly check for -1 for a missing table

[Marcus Meissner]

On Wed, Apr 16, 2008 at 11:48:31AM -0500, James Hawkins wrote:
> Hi,
> 
> Changelog:
> * Explicitly check for -1 for a missing table.
> 
>  dlls/fusion/assembly.c |    4 ++--
>  1 files changed, 2 insertions(+), 2 deletions(-)
> 
> -- 
> James Hawkins

> diff --git a/dlls/fusion/assembly.c b/dlls/fusion/assembly.c
> index 024e611..eee889a 100644
> --- a/dlls/fusion/assembly.c
> +++ b/dlls/fusion/assembly.c
> @@ -416,7 +416,7 @@ HRESULT assembly_get_name(ASSEMBLY *assembly, LPSTR *name)
>      ULONG offset;
>  
>      offset = assembly->tables[0x20].offset; /* FIXME: add constants */
> -    if (offset < 0)
> +    if (offset == -1)
>          return E_FAIL;
>  
>      asmtbl = (ASSEMBLYTABLE *)assembly_data_offset(assembly, offset);
> @@ -527,7 +527,7 @@ HRESULT assembly_get_pubkey_token(ASSEMBLY *assembly, LPSTR *token)
>      *token = NULL;
>  
>      offset = assembly->tables[0x20].offset; /* FIXME: add constants */
> -    if (offset < 0)
> +    if (offset == -1)
>          return E_FAIL;

You should make offset "signed", or pass errors down in another way.
This will not work this way.

Ciao, Marcus