ProcursusTeam / ldid

Link Identity Editor. Put real or fake signatures in a Mach-O.
GNU Affero General Public License v3.0
189 stars 45 forks source link

Added better error handling #13

Closed emilrl-gh closed 2 years ago

emilrl-gh commented 2 years ago

Started replacing asserts with if statements to check for errors

CRKatri commented 2 years ago

Thanks for getting started on this.

emilrl-gh commented 2 years ago

@CRKatri Should there be a version bump for Ldid once error checking is done?

CRKatri commented 2 years ago

Should there be a version bump for Ldid once error checking is done?

That is the current plan.

demhademha commented 2 years ago

We should ideally use exit(EXIT_FAILURE);

demhademha commented 2 years ago

And EXIT_SUCCESS

emilrl-gh commented 2 years ago

Should there be a version bump for Ldid once error checking is done?

That is the current plan.

What should the new version number be do you think?

CRKatri commented 2 years ago

Should there be a version bump for Ldid once error checking is done?

That is the current plan.

What should the new version number be do you think?

It will be 2.1.5-procursus3. The first number is the upstream version number from git.saurik.com/ldid.git. Then we do -procursus* to show the Procursus patch version.