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

Add insert support for Catalina #32

Merged
merged 1 commit into from
Dec 10, 2019
Merged

Add insert support for Catalina #32

merged 1 commit into from
Dec 10, 2019

Conversation

brycebostwick
Copy link
Contributor

For anyone else with SIP disabled that's still happily using tccutil in spite of #18 — there was some brief mention of this previously (in comments on #28), but it looks like the schema fo the access table has changed in Catalina.

There seem to be only a few columns added (and all of them work fine with some default values)

indirect_object_identifier_type    INTEGER,
indirect_object_identifier         TEXT,
indirect_object_code_identity      BLOB,
flags                              INTEGER, 
last_modified                      INTEGER NOT NULL DEFAULT (CAST(strftime('%s','now') AS INTEGER)

Tested the change with macOS 10.15.1 and it works well — based on the comments in #28 (where the error message called out an addition of 5 unexpected columns, back in March) I'm optimistic this is generally applicable to 10.15.0 as well.

Brief summary of changes:

  • Add a new case within insert_client for macOS 10.15+ that handles the above five columns
  • Add a new accessTableDigest check specific to macOS 10.15+
  • Bump util_version

Copy link
Owner

@jacobsalmela jacobsalmela left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Cool. Worked for me.

@jacobsalmela jacobsalmela merged commit cb4eed2 into jacobsalmela:master Dec 10, 2019
@dwoz
Copy link

dwoz commented Feb 19, 2024

There could be some kind of work around if we were table to grant full disk access pragmatically:

https://forums.developer.apple.com/forums/thread/670923

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.

3 participants