-2

関数がスタック変数のアドレスを返すため、通常はクラッシュという形で、意図しないプログラムの動作が発生します。次の関数は、スタック アドレスを返します。

int init(char *device, DriverType driver)
{
    int rv = -1;

    if (autodetect) {
        void *md;
        const char *p = NULL;
        char buf[PATH_MAX];

        *device = 0;
        md = discover_media_devices();
        if (!md) {
            fprintf (stderr, "open: Failed to open \"auto\" device");
            if (*device)
                fprintf (stderr, " at %s\n", device);
            else
                fprintf (stderr, "\n");
            goto failure;
        }

        while (1) {
            p = get_associated_device(md, p, MEDIA_V4L_RADIO, NULL, NONE);
            if (!p)
                break;
            snprintf(buf, sizeof(buf), "/dev/%s", p);
            device = &buf[0];
        }

    free_media_devices(md);
    /* out_of_scope: Variable "buf" goes out of scope */
    }

    switch (driver) {
            case DRIVER_ANY:
            case DRIVER_V4L2:
            default:
                    goto try_v4l2;
            case DRIVER_V4L1:
                    goto try_v4l1;
    }

try_v4l1:
    dev = v4l1_radio_dev_new();
    /* use_invalid: Using "device", which points to an out-of-scope variable "buf" */
    rv = dev->init (dev, device);
    ----------------------------

try_v4l2:
    dev = v4l2_radio_dev_new();
    /* use_invalid: Using "device", which points to an out-of-scope variable "buf" */
    rv = dev->init (dev, device);
    ----------------------------

failure:
    return rv;
}

この問題をコードで解決するのを手伝ってください

4

1 に答える 1

2

大まかに次の 2 つのオプションがあります。

  1. init関数を呼び出す前に、スタックに char を割り当てます。

    char ch[PATH_MAX];
    init (ch, ...);
    
  2. mallocを使用して関数内で char を割り当て、 init関数の外側で割り当てられたメモリを解放します。

    int init(char *device, DriverType driver)
    {
         /*...*/
         device = malloc(PATH_MAX);
         /*...*/
    }
    
    
    char* p;
    init (p, ...);
    free(p);
    

最初のオプションは、よりエレガントで効率的です。

于 2013-11-06T09:45:56.130 に答える