diff options
Diffstat (limited to 'kernel/bpf/arraymap.c')
| -rw-r--r-- | kernel/bpf/arraymap.c | 10 |
1 files changed, 8 insertions, 2 deletions
diff --git a/kernel/bpf/arraymap.c b/kernel/bpf/arraymap.c index 3f4c99e06c6b..b0799bced518 100644 --- a/kernel/bpf/arraymap.c +++ b/kernel/bpf/arraymap.c | |||
| @@ -28,11 +28,17 @@ static struct bpf_map *array_map_alloc(union bpf_attr *attr) | |||
| 28 | attr->value_size == 0) | 28 | attr->value_size == 0) |
| 29 | return ERR_PTR(-EINVAL); | 29 | return ERR_PTR(-EINVAL); |
| 30 | 30 | ||
| 31 | if (attr->value_size >= 1 << (KMALLOC_SHIFT_MAX - 1)) | ||
| 32 | /* if value_size is bigger, the user space won't be able to | ||
| 33 | * access the elements. | ||
| 34 | */ | ||
| 35 | return ERR_PTR(-E2BIG); | ||
| 36 | |||
| 31 | elem_size = round_up(attr->value_size, 8); | 37 | elem_size = round_up(attr->value_size, 8); |
| 32 | 38 | ||
| 33 | /* check round_up into zero and u32 overflow */ | 39 | /* check round_up into zero and u32 overflow */ |
| 34 | if (elem_size == 0 || | 40 | if (elem_size == 0 || |
| 35 | attr->max_entries > (U32_MAX - sizeof(*array)) / elem_size) | 41 | attr->max_entries > (U32_MAX - PAGE_SIZE - sizeof(*array)) / elem_size) |
| 36 | return ERR_PTR(-ENOMEM); | 42 | return ERR_PTR(-ENOMEM); |
| 37 | 43 | ||
| 38 | array_size = sizeof(*array) + attr->max_entries * elem_size; | 44 | array_size = sizeof(*array) + attr->max_entries * elem_size; |
| @@ -105,7 +111,7 @@ static int array_map_update_elem(struct bpf_map *map, void *key, void *value, | |||
| 105 | /* all elements already exist */ | 111 | /* all elements already exist */ |
| 106 | return -EEXIST; | 112 | return -EEXIST; |
| 107 | 113 | ||
| 108 | memcpy(array->value + array->elem_size * index, value, array->elem_size); | 114 | memcpy(array->value + array->elem_size * index, value, map->value_size); |
| 109 | return 0; | 115 | return 0; |
| 110 | } | 116 | } |
| 111 | 117 | ||
