facebookarchive/caffe2

Segfaults if GivenTensorFill shape doesn't match actual floats

Open

#1 426 ouverte le 3 nov. 2017

Voir sur GitHub
 (0 commentaires) (0 réactions) (0 assignés)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

Guide contributeur

Stack technique
cpp
Domaine
datamachine learning
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.
3
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
half day
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
fresh
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
clear
Prérequis
C++Caffe2 operator internals
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
30
Direction de recherche
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.