Fix debug print in intel.c not using the new struct layout
Let the user define whether a mem event listener is required for the domain to run
Only should need PARTIAL for physical dump.
Don't return VMI_SUCCESS by default.
Merge pull request #199 from awsaba/minor_fixes

A few Minor fixes
Merge pull request #197 from tklengyel/debug_fix

Fix debug print in intel.c not using the new struct layout
Merge pull request #198 from tklengyel/event_listener_required

Let the user define whether a mem event listener is required for the dom...
Coverity fixes in core.c

CID 703181 (#1 of 1): Dereference before null check (REVERSE_INULL)check_after_deref: Null-checking vmi suggests that it may be null, but it has already been dereferenced on all paths leading to the check.
CID 1237203 (#1 of 1): Unchecked return value (CHECKED_RETURN)3. check_return: Calling arch_init without checking return value (as is done elsewhere 7 out of 8 times)
Coverity fixes in arch/intel.c

CID 703167 (#1 of 1): Missing parentheses (CONSTANT_EXPRESSION_RESULT)missing_parentheses: !instance->os_type == VMI_OS_WINDOWS is always false regardless of the values of its operands. This occurs as the logical operand of if.
CID 1237204 (#1 of 1): Unchecked return value (CHECKED_RETURN)2. check_return: Calling vmi_read_64_pa without checking return value (as is done elsewhere 14 out of 15 times).
CID 1237206 (#1 of 1): Logically dead code (DEADCODE)dead_error_line: Execution cannot reach this statement: return;.
Coverity fix in driver/xen.c

CID 703184 (#1 of 1): Dereference before null check (REVERSE_INULL)check_after_deref: Null-checking name suggests that it may be null, but it has already been dereferenced on all paths leading to the check.
Kdbg search resource leak fixes

Coverity ID 1237209: Unsigned compared against 0 (NO_EFFECT)unsigned_compare: This greater-than-or-equal-to-zero comparison of an unsigned value is always true. page_paddr + step >= 0UL
Coverity ID 1237202: Unchecked return value (CHECKED_RETURN)7. check_return: Calling driver_get_vcpureg without checking return value (as is done elsewhere 13 out of 16 times).
Coverity ID 1237215: Resource leak (RESOURCE_LEAK)10. leaked_storage: Variable bm64 going out of scope leaks the storage it points to.
Coverity ID 1237216: Resource leak (RESOURCE_LEAK)16. leaked_storage: Variable bm32 going out of scope leaks the storage it points to.
Coverity ID 1237218: Resource leak (RESOURCE_LEAK)8. leaked_storage: Variable bm going out of scope leaks the storage it points to.
Fix Coverity errors in driver/xen_events.c

Coverity ID 1237221: Uninitialized scalar variable (UNINIT)11. uninit_use: Using uninitialized value out_access.
Coverity ID 1237223 (#1 of 1): Structurally dead code (UNREACHABLE)unreachable: This code cannot be reached: errdone:
Coverity fixes in os/linux

CID 1237208 (#1 of 1): Test should be assignment (NO_EFFECT)no_effect_test: The boolean test in statement vmi->page_mode == VMI_PM_IA32E; has no effect.
CID 1237214 (#1 of 1): Resource leak (RESOURCE_LEAK)10. leaked_storage: Variable f going out of scope leaks the storage it points to.
Lots of clang warning fixes
Coverity fixes in examples

CID 703168 (#1 of 1): Logically dead code (DEADCODE)dead_error_line: Execution cannot reach this statement: free(procname);.
CID 703195 (#1 of 1): Uninitialized scalar variable (UNINIT)5. uninit_use: Using uninitialized value next_module.
CID 703196 (#1 of 1): Uninitialized scalar variable (UNINIT)11. uninit_use: Using uninitialized value name_offset.
CID 703198 (#1 of 1): Uninitialized scalar variable (UNINIT)11. uninit_use: Using uninitialized value pid_offset.
CID 703199 (#1 of 1): Uninitialized scalar variable (UNINIT)11. uninit_use: Using uninitialized value tasks_offset.
CID 1237217 (#1 of 1): Resource leak (RESOURCE_LEAK)10. leaked_storage: Variable pdb_header going out of scope leaks the storage it points to.
Coverity fix with vmi_get_bit

CID 703173 (#1 of 1): Unintentional integer overflow (OVERFLOW_BEFORE_WIDEN)overflow_before_widen: Potentially overflowing expression 1 << bit with type int (32 bits, signed) is evaluated using 32-bit arithmetic, and then used in a context that expects an expression of type reg_t (64 bits, unsigned).
Merge pull request #200 from tklengyel/coverity_fixes2

Fix Coverity errors in driver/xen_events.c
Merge pull request #201 from tklengyel/coverity_fixes3

Kdbg search resource leak fixes
Merge pull request #202 from tklengyel/coverity_fixes4

Coverity fixes in core.c
Merge pull request #203 from tklengyel/coverity_fixes5

Coverity fixes in arch/intel.c