diff options
author | Alan Coopersmith <alan.coopersmith@oracle.com> | 2011-12-01 23:45:42 -0800 |
---|---|---|
committer | Alan Coopersmith <alan.coopersmith@oracle.com> | 2011-12-05 14:33:00 -0800 |
commit | e4dcf580f007d6adcf5b0c639865d7aaab1a319d (patch) | |
tree | 856df97bdb033f286b6767ee2aaf9abd44708b91 /hw/xnest/Display.c | |
parent | 8b6a75009746db4b8e2e180210ac097e35136bcd (diff) |
LoaderOpen returns either a valid pointer or NULL, so don't check for < 0
Fixes Sun cc warning that was recently elevated to error by the
stricter default CFLAGS changes to xorg-macros:
"loadmod.c", line 914: improper pointer/integer combination: op "<"
Should have been changed when commit ab7f057ce9df4e905b12 changed the
LoaderOpen return type from int to void *.
Changes log message when file is found but dlopen() fails from:
(EE) LoadModule: Module dbe does not have a dbeModuleData data object.
(EE) Failed to load module "dbe" (invalid module, 0)
to:
(EE) Failed to load module "dbe" (loader failed, 7)
Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
Reviewed-by: Adam Jackson <ajax@redhat.com>
Diffstat (limited to 'hw/xnest/Display.c')
0 files changed, 0 insertions, 0 deletions