Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

dnn: Fix dangling pointers returned by GetLayerNames #927

Open
wants to merge 2 commits into
base: dev
Choose a base branch
from
Open

dnn: Fix dangling pointers returned by GetLayerNames #927

wants to merge 2 commits into from

Commits on Jul 21, 2022

  1. dnn: Fix dangling pointers returned by GetLayerNames 

    'GetLayerNames' returns an array of 'char' pointers to cstrings
in a 'vector<string>'; unfortunately, once the vector is out of
scope, the strings are destroyed. 'GetLayerNames' callers are then
left with dangling pointers.

This change fixes the problem by expanding the 'strs' buffer
returned by 'GetLayerNames' and copying the vector's cstrings into it.
    @deradiri
    deradiri committed Jul 21, 2022
    Configuration menu
    Copy the full SHA
    ed54000 View commit details
    Browse the repository at this point in the history

  2. openvino: Make checkNet verify multiple layers

    @deradiri
    deradiri committed Jul 21, 2022
    Configuration menu
    Copy the full SHA
    b55ebc1 View commit details
    Browse the repository at this point in the history