code-423n4 / 2023-09-ondo-findings

7 stars 5 forks source link

QA Report #545

Closed c4-submissions closed 1 year ago

c4-submissions commented 1 year ago

See the markdown file with the details of this report here.

c4-pre-sort commented 1 year ago

raymondfam marked the issue as sufficient quality report

c4-judge commented 1 year ago

kirk-baird marked the issue as grade-c

kirk-baird commented 1 year ago

1) Only exists in admin only function and so is only an issue if the owner is adding a very large number of users. I don't see where the number 15 is retrieved from this seems extremely low.

2) Using an array here is fine

3) Admin role vs owner role makes no difference

4) Is a valid issue

3 out of 4 issues are invalid or irrelevant marking this as unsatisfactory.