gocodebox / gocodebox/lifterlms

Fatal error with a large attempt key

Open
#2,746 0 comments 0 reactions 1 assignee Claimed by @ideadude View on GitHub
Dominant language
PHP
Stars
212
Forks
140
Avg merge
2d 14h
Merged PRs (30d)
19

Description

### Reproduction Steps

1. Visit the page for the results of a quiz
2. Change `attempt_key=` to `attempt_key=999999999999999999999`

### Expected Behavior

Shows quiz information but no quiz attempt details.

### Actual Behavior

Fatal error

### Error Messages / Logs

```
Fatal error: Uncaught DivisionByZeroError: Modulo by zero in /srv/users/manager/apps/hogg-new/public/wp-content/plugins/lifterlms/includes/class.llms.hasher.php:136 Stack trace: #0 /srv/users/manager/apps/hogg-new/public/wp-content/plugins/lifterlms/includes/class.llms.hasher.php(221): LLMS_Hasher::mod(0, 4.3674252383914E+37) #1 /srv/users/manager/apps/hogg-new/public/wp-content/plugins/lifterlms/includes/models/model.llms.student.quizzes.php(335): LLMS_Hasher::unhash('999999999999999...') #2 /srv/users/manager/apps/hogg-new/public/wp-content/plugins/lifterlms/includes/models/model.llms.student.quizzes.php(155): LLMS_Student_Quizzes->parse_attempt_key('999999999999999...') #3 /srv/users/manager/apps/hogg-new/public/wp-content/plugins/lifterlms/templates/quiz/results.php(40): LLMS_Student_Quizzes->get_attempt_by_key('999999999999999...') #4 /srv/users/manager/apps/hogg-new/public/wp-content/plugins/lifterlms/includes/functions/llms.functions.template.php(120): include('/srv/users/mana...') #5 /srv/users/manager/apps/hogg-new/public/wp-content/plugins/lifterlms/includes/functions/llms.functions.templates.quizzes.php(162): llms_get_template('quiz/results.ph...') #6 /srv/users/manager/apps/hogg-new/public/wp-includes/class-wp-hook.php(324): lifterlms_template_quiz_results('') #7 /srv/users/manager/apps/hogg-new/public/wp-includes/class-wp-hook.php(348): WP_Hook->apply_filters(NULL, Array) #8 /srv/users/manager/apps/hogg-new/public/wp-includes/plugin.php(517): WP_Hook->do_action(Array) #9 /srv/users/manager/apps/hogg-new/public/wp-content/plugins/lifterlms/templates/content-single-quiz-before.php(13): do_action('lifterlms_singl...') #10 /srv/users/manager/apps/hogg-new/public/wp-includes/template.php(812): require('/srv/users/mana...') #11 /srv/users/manager/apps/hogg-new/public/wp-content/plugins/lifterlms/includes/functions/llms-functions-content.php(52): load_template('/srv/users/mana...', false) #12 /srv/users/manager/apps/hogg-new/public/wp-includes/class-wp-hook.php(324): llms_get_post_content('

Test quiz he...') #13 /srv/users/manager/apps/hogg-new/public/wp-includes/plugin.php(205): WP_Hook->apply_filters('

Test quiz he...', Array) #14 /srv/users/manager/apps/hogg-new/public/wp-includes/blocks/post-content.php(50): apply_filters('the_content', '

Test quiz he...') #15 /srv/users/manager/apps/hogg-new/public/wp-includes/class-wp-block.php(519): render_block_core_post_content(Array, '

Test quiz he...', Object(WP_Block)) #16 /srv/users/manager/apps/hogg-new/public/wp-includes/class-wp-block.php(499): WP_Block->render() #17 /srv/users/manager/apps/hogg-new/public/wp-includes/blocks.php(2061): WP_Block->render() #18 /srv/users/manager/apps/hogg-new/public/wp-includes/blocks.php(2113): render_block(Array) #19 /srv/users/manager/apps/hogg-new/public/wp-includes/block-template.php(260): do_blocks('

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.