Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Improved support for Access 2010, 2013, and 2016 #9

Merged
merged 1 commit into from
Dec 28, 2018
Merged

Improved support for Access 2010, 2013, and 2016 #9

merged 1 commit into from
Dec 28, 2018

Conversation

evanmiller
Copy link

Some Access 2010 files use 0x03 as the version number rather than 0x0103. For this reason I have changed the call to mdb_get_int32 to mdb_get_byte.

In addition, according to the Library of Congress page:

https://www.loc.gov/preservation/digital/formats/fdd/fdd000463.shtml

Access 2016 uses 0x05 as the version number. I have inferred from the Wikipedia page that Access 2013 likely uses 0x04.

Some Access 2010 files use 0x03 as the version number rather than
0x0103. For this reason I have changed the call to mdb_get_int32 to
mdb_get_byte.

In addition, according to the Library of Congress page:

https://www.loc.gov/preservation/digital/formats/fdd/fdd000463.shtml

Access 2016 uses 0x05 as the version number. I have inferred from the
Wikipedia page that Access 2013 likely uses 0x04.
@evanmiller
Copy link
Author

Bump

@cyberemissary
Copy link
Owner

Have you tested these with corresponding versions of access files? are the results as you expect?

@evanmiller
Copy link
Author

I have tested with Access 2010 files that do not currently open with mdbtools.

The linked Library of Congress page indicates that Access 2016 has the same file format except adding large integer support. I have not tested against these files, but the provided patch is meant simply to identify them.

@cyberemissary cyberemissary merged commit 9ca5f6e into cyberemissary:master Dec 28, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants