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

False Positive (ForceOP) #22

Closed
4 of 5 tasks
OpticFusion1 opened this issue Mar 17, 2019 · 13 comments
Closed
4 of 5 tasks

False Positive (ForceOP) #22

OpticFusion1 opened this issue Mar 17, 2019 · 13 comments

Comments

@OpticFusion1
Copy link
Owner

OpticFusion1 commented Mar 17, 2019

General Troubleshooting

  • I have checked for similar issues.
  • I have updated to the latest version.
  • I have checked the branches or the maintainers' PRs for upcoming features/bug fixes.

Issue

Issue Type

  • Bug Report
  • Feature Request

Description

Certain plugins cause a false positive due Player#setOp
this is usually used in the following way
setOp
dispatchCommand
setOp

The plugin below is an example of this, more plugins with this false positive can be provided as needed
ActionLibOpenSource(47507).zip

Repository owner deleted a comment from MrEMan Mar 23, 2019
@OpticFusion1
Copy link
Owner Author

OpticFusion1 commented Mar 23, 2019

for this issue to be considered closed, NONE of these and ANY plugin using these methods should be logged as a false positive, while actual force ops are

@ghost
Copy link

ghost commented Mar 24, 2019

That might be the most difficult issue to fix

@ghost
Copy link

ghost commented Mar 24, 2019

@OpticFusion1 The false positive comes from

p.setOp not if(p.isOp)

@OpticFusion1
Copy link
Owner Author

@OpticFusion1 The false positive comes from

p.setOp not if(p.isOp)

I'm aware :P
If we can fix this specific false positive then we will have MOST of not all false positives dealt with

@OpticFusion1
Copy link
Owner Author

@OpticFusion1
Copy link
Owner Author

AdminTools(14781).zip

@OpticFusion1
Copy link
Owner Author

DeACoudre(14635).zip

@OpticFusion1
Copy link
Owner Author

@OpticFusion1
Copy link
Owner Author

gonna try and spend a few hours on this issue, it's the biggest issue currently, besides dealing with obfuscated malware..

@OpticFusion1
Copy link
Owner Author

There's been quite a few updates since this, closing. A general issue will be made if a false-positive of this type is found again (however there's definitely going to be another issue opened up due to this definitely not getting fixed)

@OpticFusion1
Copy link
Owner Author

None of these throw ForceOP triggers anymore which is nice.
AntiGriefAntiGrief(14593).jar triggers the AntiCrash and Bewerbung checks
CommandsXL(12336).jar triggers the Annoyance check
DungeonsXL(9488).jar triggers the Annoyance check
LibsCommands(88).jar triggers the AdminTools check

I just can't win can i XD

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

No branches or pull requests

1 participant