Konstantin Khlebnikov | 2d6f45b | 2016-03-17 14:22:08 -0700 | [diff] [blame] | 1 | /* |
| 2 | * Regression3 |
| 3 | * Description: |
| 4 | * Helper radix_tree_iter_retry resets next_index to the current index. |
| 5 | * In following radix_tree_next_slot current chunk size becomes zero. |
| 6 | * This isn't checked and it tries to dereference null pointer in slot. |
| 7 | * |
Matthew Wilcox | 148deab | 2016-12-14 15:08:49 -0800 | [diff] [blame] | 8 | * Helper radix_tree_iter_resume reset slot to NULL and next_index to index + 1, |
Konstantin Khlebnikov | 7475851 | 2016-03-17 14:22:11 -0700 | [diff] [blame] | 9 | * for tagger iteraction it also must reset cached tags in iterator to abort |
| 10 | * next radix_tree_next_slot and go to slow-path into radix_tree_next_chunk. |
| 11 | * |
Konstantin Khlebnikov | 2d6f45b | 2016-03-17 14:22:08 -0700 | [diff] [blame] | 12 | * Running: |
| 13 | * This test should run to completion immediately. The above bug would |
| 14 | * cause it to segfault. |
| 15 | * |
| 16 | * Upstream commit: |
| 17 | * Not yet |
| 18 | */ |
| 19 | #include <linux/kernel.h> |
| 20 | #include <linux/gfp.h> |
| 21 | #include <linux/slab.h> |
| 22 | #include <linux/radix-tree.h> |
| 23 | #include <stdlib.h> |
| 24 | #include <stdio.h> |
| 25 | |
| 26 | #include "regression.h" |
| 27 | |
| 28 | void regression3_test(void) |
| 29 | { |
| 30 | RADIX_TREE(root, GFP_KERNEL); |
Konstantin Khlebnikov | 7475851 | 2016-03-17 14:22:11 -0700 | [diff] [blame] | 31 | void *ptr0 = (void *)4ul; |
| 32 | void *ptr = (void *)8ul; |
Konstantin Khlebnikov | 2d6f45b | 2016-03-17 14:22:08 -0700 | [diff] [blame] | 33 | struct radix_tree_iter iter; |
| 34 | void **slot; |
| 35 | bool first; |
| 36 | |
Rehas Sachdeva | 73bc029 | 2017-01-04 11:55:00 -0500 | [diff] [blame] | 37 | printv(1, "running regression test 3 (should take milliseconds)\n"); |
Konstantin Khlebnikov | 2d6f45b | 2016-03-17 14:22:08 -0700 | [diff] [blame] | 38 | |
Konstantin Khlebnikov | 7475851 | 2016-03-17 14:22:11 -0700 | [diff] [blame] | 39 | radix_tree_insert(&root, 0, ptr0); |
Konstantin Khlebnikov | 2d6f45b | 2016-03-17 14:22:08 -0700 | [diff] [blame] | 40 | radix_tree_tag_set(&root, 0, 0); |
| 41 | |
| 42 | first = true; |
| 43 | radix_tree_for_each_tagged(slot, &root, &iter, 0, 0) { |
Rehas Sachdeva | 73bc029 | 2017-01-04 11:55:00 -0500 | [diff] [blame] | 44 | printv(2, "tagged %ld %p\n", iter.index, *slot); |
Konstantin Khlebnikov | 2d6f45b | 2016-03-17 14:22:08 -0700 | [diff] [blame] | 45 | if (first) { |
| 46 | radix_tree_insert(&root, 1, ptr); |
| 47 | radix_tree_tag_set(&root, 1, 0); |
| 48 | first = false; |
| 49 | } |
| 50 | if (radix_tree_deref_retry(*slot)) { |
Rehas Sachdeva | 73bc029 | 2017-01-04 11:55:00 -0500 | [diff] [blame] | 51 | printv(2, "retry at %ld\n", iter.index); |
Konstantin Khlebnikov | 2d6f45b | 2016-03-17 14:22:08 -0700 | [diff] [blame] | 52 | slot = radix_tree_iter_retry(&iter); |
| 53 | continue; |
| 54 | } |
| 55 | } |
| 56 | radix_tree_delete(&root, 1); |
| 57 | |
| 58 | first = true; |
| 59 | radix_tree_for_each_slot(slot, &root, &iter, 0) { |
Rehas Sachdeva | 73bc029 | 2017-01-04 11:55:00 -0500 | [diff] [blame] | 60 | printv(2, "slot %ld %p\n", iter.index, *slot); |
Konstantin Khlebnikov | 2d6f45b | 2016-03-17 14:22:08 -0700 | [diff] [blame] | 61 | if (first) { |
| 62 | radix_tree_insert(&root, 1, ptr); |
| 63 | first = false; |
| 64 | } |
| 65 | if (radix_tree_deref_retry(*slot)) { |
Rehas Sachdeva | 73bc029 | 2017-01-04 11:55:00 -0500 | [diff] [blame] | 66 | printv(2, "retry at %ld\n", iter.index); |
Konstantin Khlebnikov | 2d6f45b | 2016-03-17 14:22:08 -0700 | [diff] [blame] | 67 | slot = radix_tree_iter_retry(&iter); |
| 68 | continue; |
| 69 | } |
| 70 | } |
| 71 | radix_tree_delete(&root, 1); |
| 72 | |
| 73 | first = true; |
| 74 | radix_tree_for_each_contig(slot, &root, &iter, 0) { |
Rehas Sachdeva | 73bc029 | 2017-01-04 11:55:00 -0500 | [diff] [blame] | 75 | printv(2, "contig %ld %p\n", iter.index, *slot); |
Konstantin Khlebnikov | 2d6f45b | 2016-03-17 14:22:08 -0700 | [diff] [blame] | 76 | if (first) { |
| 77 | radix_tree_insert(&root, 1, ptr); |
| 78 | first = false; |
| 79 | } |
| 80 | if (radix_tree_deref_retry(*slot)) { |
Rehas Sachdeva | 73bc029 | 2017-01-04 11:55:00 -0500 | [diff] [blame] | 81 | printv(2, "retry at %ld\n", iter.index); |
Konstantin Khlebnikov | 2d6f45b | 2016-03-17 14:22:08 -0700 | [diff] [blame] | 82 | slot = radix_tree_iter_retry(&iter); |
| 83 | continue; |
| 84 | } |
| 85 | } |
| 86 | |
Konstantin Khlebnikov | 7475851 | 2016-03-17 14:22:11 -0700 | [diff] [blame] | 87 | radix_tree_for_each_slot(slot, &root, &iter, 0) { |
Rehas Sachdeva | 73bc029 | 2017-01-04 11:55:00 -0500 | [diff] [blame] | 88 | printv(2, "slot %ld %p\n", iter.index, *slot); |
Konstantin Khlebnikov | 7475851 | 2016-03-17 14:22:11 -0700 | [diff] [blame] | 89 | if (!iter.index) { |
Rehas Sachdeva | 73bc029 | 2017-01-04 11:55:00 -0500 | [diff] [blame] | 90 | printv(2, "next at %ld\n", iter.index); |
Matthew Wilcox | 148deab | 2016-12-14 15:08:49 -0800 | [diff] [blame] | 91 | slot = radix_tree_iter_resume(slot, &iter); |
Konstantin Khlebnikov | 7475851 | 2016-03-17 14:22:11 -0700 | [diff] [blame] | 92 | } |
| 93 | } |
| 94 | |
| 95 | radix_tree_for_each_contig(slot, &root, &iter, 0) { |
Rehas Sachdeva | 73bc029 | 2017-01-04 11:55:00 -0500 | [diff] [blame] | 96 | printv(2, "contig %ld %p\n", iter.index, *slot); |
Konstantin Khlebnikov | 7475851 | 2016-03-17 14:22:11 -0700 | [diff] [blame] | 97 | if (!iter.index) { |
Rehas Sachdeva | 73bc029 | 2017-01-04 11:55:00 -0500 | [diff] [blame] | 98 | printv(2, "next at %ld\n", iter.index); |
Matthew Wilcox | 148deab | 2016-12-14 15:08:49 -0800 | [diff] [blame] | 99 | slot = radix_tree_iter_resume(slot, &iter); |
Konstantin Khlebnikov | 7475851 | 2016-03-17 14:22:11 -0700 | [diff] [blame] | 100 | } |
| 101 | } |
| 102 | |
| 103 | radix_tree_tag_set(&root, 0, 0); |
| 104 | radix_tree_tag_set(&root, 1, 0); |
| 105 | radix_tree_for_each_tagged(slot, &root, &iter, 0, 0) { |
Rehas Sachdeva | 73bc029 | 2017-01-04 11:55:00 -0500 | [diff] [blame] | 106 | printv(2, "tagged %ld %p\n", iter.index, *slot); |
Konstantin Khlebnikov | 7475851 | 2016-03-17 14:22:11 -0700 | [diff] [blame] | 107 | if (!iter.index) { |
Rehas Sachdeva | 73bc029 | 2017-01-04 11:55:00 -0500 | [diff] [blame] | 108 | printv(2, "next at %ld\n", iter.index); |
Matthew Wilcox | 148deab | 2016-12-14 15:08:49 -0800 | [diff] [blame] | 109 | slot = radix_tree_iter_resume(slot, &iter); |
Konstantin Khlebnikov | 7475851 | 2016-03-17 14:22:11 -0700 | [diff] [blame] | 110 | } |
| 111 | } |
| 112 | |
Konstantin Khlebnikov | 2d6f45b | 2016-03-17 14:22:08 -0700 | [diff] [blame] | 113 | radix_tree_delete(&root, 0); |
| 114 | radix_tree_delete(&root, 1); |
| 115 | |
Rehas Sachdeva | 73bc029 | 2017-01-04 11:55:00 -0500 | [diff] [blame] | 116 | printv(1, "regression test 3 passed\n"); |
Konstantin Khlebnikov | 2d6f45b | 2016-03-17 14:22:08 -0700 | [diff] [blame] | 117 | } |