Skip to content
New issue

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

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

Already on GitHub? # to your account

fix: fix incorrect nil return value #6079

Merged
merged 1 commit into from
Mar 3, 2025

Conversation

huochexizhan
Copy link
Contributor

@huochexizhan huochexizhan commented Feb 28, 2025

Proposed changes

Since we have already checked err before and returned != nil, err must be nil here. In fact, it should returnqerr .

Checklist

  • Pull request is created against the dev branch
  • All checks passed (lint, unit/integration/regression tests etc.) with my changes
  • I have added tests that prove my fix is effective or that my feature works
  • I have added necessary documentation (if appropriate)

Signed-off-by: huochexizhan <huochexizhan@outlook.com>
@auto-assign auto-assign bot requested a review from dogancanbakir February 28, 2025 17:42
@dogancanbakir dogancanbakir requested a review from ehsandeep March 3, 2025 09:47
@ehsandeep ehsandeep merged commit 3a3f5e2 into projectdiscovery:dev Mar 3, 2025
19 checks passed
@ehsandeep ehsandeep removed their request for review March 3, 2025 10:49
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants