facebookarchive/caffe2

Segfaults if GivenTensorFill shape doesn't match actual floats

Open

#1.426 aberto em 3 de nov. de 2017

Ver no GitHub
 (0 comments) (0 reactions) (0 assignees)Shell (8.424 stars) (1.954 forks)batch import
bughelp wanted

Description

It shouldn't do that.

Repro steps:

  1. Make a GivenTensorFill. Set its shape to something big, give it only one value in floats
  2. Run it

Expected result: validation failure Actual result: segfault

Guia do colaborador

Pilha de tecnologia
cpp
Domain
datamachine learning
Tipo Issue
bug
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
3
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
half day
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
fresh
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
clear
Prerequisites
C++Caffe2 operator internals
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
30
Direção de pesquisa
The issue is that GivenTensorFill segfaults when the specified shape does not match the number of provided float values. The fix should add validation to check that the total size of the shape equals the length of the floats array before executing the fill. Look for the GivenTensorFill operator implementation in the Caffe2 source code, likely in caffe2/operators/utility ops.cc or similar. Add a check that returns an error instead of allowing the segfault.