alrevuelta / cONNXr

Pure C ONNX runtime with zero dependancies for embedded devices
MIT License
192 stars 31 forks source link

Two missing null check for return values of searchAttributeNyName() #102

Open szsam opened 1 year ago

szsam commented 1 year ago

The result of these calls to searchAttributeNyName is not checked for null, but 92% of calls to searchAttributeNyName check for null. https://github.com/alrevuelta/cONNXr/blob/7108b7b9003b3614afd1a80b8e0ea8337a9c3a89/src/operators/ai.onnx/Constant/12/prepare_operator__ai_onnx__constant__12.c#L19 https://github.com/alrevuelta/cONNXr/blob/7108b7b9003b3614afd1a80b8e0ea8337a9c3a89/src/operators/ai.onnx/MaxPool/12/prepare_operator__ai_onnx__maxpool__12.c#L26