facebookarchive/caffe2

Segfaults if GivenTensorFill shape doesn't match actual floats

Aperta

#1426 aperta il 3 nov 2017

Vedi su GitHubGuida contributor
 (0 commenti) (0 reazioni) (0 assegnatari)Shell (1954 fork)batch import
bughelp wanted

Metriche repository

Star
 (8424 stelle)
Metriche merge PR
 (Nessuna PR mergiata in 30 g)

Descrizione

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

Guida contributor

Direzione di ricerca
Il problema è che GivenTensorFill causa un segfault quando la forma specificata non corrisponde al numero di valori float forniti. La correzione dovrebbe aggiungere una convalida per verificare che la dimensione totale della forma sia uguale alla lunghezza dell'array di float prima di eseguire il riempimento. Cerca l'implementazione dell'operatore GivenTensorFill nel codice sorgente di Caffe2, probabilmente in caffe2/operators/utility ops.cc o simile. Aggiungi un controllo che restituisca un errore invece di consentire il segfault.
Tech stack
cpppython
Dominio
machine learningai
Tipo issue
Bug
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-3 ore
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Nuova
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
C++
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
50