vanhauser-thc/thc-hydra

[BUG] When resuming a session, finished tries count towards new session tries - messing up ETA

Open

#824 ouverte le 19 févr. 2023

Voir sur GitHub
 (0 commentaires) (0 réactions) (0 assignés)C (1 913 forks)batch import
best practicegood first issuehelp wanted

Métriques du dépôt

Stars
 (8 689 stars)
Métriques de merge PR
 (Merge moyen 2j 14h) (2 PRs mergées en 30 j)

Description

I left a session running for about 10 minutes during which ~9000 tries were done. Then I remebered to add -f option so I stopped the execution and resumed it using -R with the added option. Now after 1 minute I get: [STATUS] 10166.00 tries/min, 10166 tries in 00:01h, 989834 to do in 01:38h, 32 active even though the actual speed is somewhere around(from the previous session): [STATUS] 634.00 tries/min, 634 tries in 00:01h, 999366 to do in 26:17h, 32 active

With time due to averaging it does converge towards the actual speed but I believe this is undesired.

Guide contributeur

Direction de recherche
Examinez la logique de reprise de session dans thc hydra, probablement dans les fichiers sources principaux tels que hydra session.c ou hydra core.c. Le bogue semble être que les essais terminés d'une session précédente ne sont pas réinitialisés ou exclus lors de la reprise avec de nouvelles options, ce qui entraîne un calcul de l'ETA utilisant des compteurs d'essais gonflés. Recherchez comment le compteur 'tries' est initialisé lors de la reprise et comment le module de statistiques accumule les données au fil du temps. La correction pourrait impliquer de réinitialiser le compteur d'essais terminés ou d'ajuster correctement le calcul de l'ETA pour ne prendre en compte que les nouveaux essais.
Stack technique
c
Domaine
clisecurity
Type d'issue
Bug
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
2
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-3 heures
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
CGit
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
75