Fix issue #309 - Advance offset by string table field size

[ghost]

Fixes issue #309

There's a bug in master where the code ends up misreading the symbol table because we don't advance the offset prior to reading the strings.

This change fixes the issue by adding the correct value to the offset and also includes a unit test that covers this case.

[m4b]

this is great thank you! I think this is a longstanding bug/irritation you've fixed, much appreciated!

[m4b]

@track-5 looks like it needs a rustfmt; you can also add #[rustfmt::skip] to that test block of bytes, since it doesn't need to be formatted.

[ghost]

@m4b my bad! Just ran cargo fmt and pushed the new commit :)

[m4b]

Looks like you need to use core instead of size for sizeof

[ghost]

Done. Ran the cargo build command locally to be sure it works now:

P:\goblin>cargo build --no-default-features --features="pe32"
    Finished dev [unoptimized + debuginfo] target(s) in 0.04s

[m4b]

ok released in 0.5.2, thanks for your help!