You do know that this VirtualAddress is actually relative to the module base?Precisely what I was thinking, too. This mistake is made often actually since the struct member is named as a VA and not an RVA. Unless you read that it's relative to the actual import table one might wrongfully assume it's truly a virtual address at first glance.
Accept nothing less than STATUS_SUCCESS