3

我正在编写一个能够在屏幕上绘制像素的框架。但是现在我正在尝试更新屏幕，前 4 个像素显示随机颜色。

当我只是将一个指向图像数据的指针发送到我的窗口模块时，我没有遇到这个问题，但是现在我将一个绘图函数作为闭包传递，这开始发生了。我不认为这是我的绘图功能中的问题，因为我写了这些像素并且它们仍然被修改。

我的主要绘图功能：

// Define drawing function
fn rand_pixel(image_width: usize, image_height: usize) -> *const std::ffi::c_void
{
    // Define colors
    let black_pixel = image::Color { red: 0, green: 0, blue: 0 };
    let white_pixel = image::Color { red: 255, green: 255, blue: 255 };

    // Create blank image
    let mut pixels: image::Image = image::Image::new(image_width, image_height, black_pixel);

    // Get two random numbers
    use rand::Rng;
    let mut rng = rand::thread_rng();
    let rand_x = rng.gen::<usize>() % image_width;
    let rand_y = rng.gen::<usize>() % image_height;

    // Change pixels
    pixels.set_pixel(rand_x, rand_y, white_pixel);

    // return pointer
    pixels.get_ptr()
}

我的窗口绘图例程：

// Execute draw function
let image_ptr = draw_function();

unsafe 
{
    // Create texture
    let mut tex_id: gl::types::GLuint = 0;
    gl::GenTextures(1, &mut tex_id);
    gl::BindTexture(gl::TEXTURE_2D, tex_id);

    // Move pixels to texture
    gl::TexImage2D(gl::TEXTURE_2D, 0, gl::RGBA as i32, 
        image_width, image_height, 0, gl::RGBA, gl::UNSIGNED_BYTE, image_ptr);

    gl::TexParameteri(gl::TEXTURE_2D, gl::TEXTURE_MIN_FILTER, gl::NEAREST as i32);
    gl::TexParameteri(gl::TEXTURE_2D, gl::TEXTURE_MAG_FILTER, gl::NEAREST as i32);

    // Create read framebuffer
    let mut read_fbo_id: gl::types::GLuint = 0;
    gl::GenFramebuffers(1, &mut read_fbo_id);
    gl::BindFramebuffer(gl::READ_FRAMEBUFFER, read_fbo_id);

    // Bind texture to read framebuffer
    gl::FramebufferTexture2D(gl::READ_FRAMEBUFFER, gl::COLOR_ATTACHMENT0, gl::TEXTURE_2D, tex_id, 0);

    // Transfer from read framebuffer to draw framebuffer
    gl::BlitFramebuffer(0, 0, image_width, image_height, 
        0, 0, window_width as i32, window_height as i32, 
        gl::COLOR_BUFFER_BIT, gl::NEAREST);

    // Clear out read framebuffer
    gl::BindFramebuffer(gl::READ_FRAMEBUFFER, 0);
    gl::DeleteFramebuffers(1, &mut read_fbo_id);
};

// Update screen
let _ = windowed_context.swap_buffers();
std::thread::sleep(std::time::Duration::from_millis(15));

可以找到完整的项目： https ://github.com/Pandabear314/sam_graphics

我期待一个黑屏，每帧在屏幕上随机绘制一个白色像素。然而，左下角包括 4 个随机像素。第一个和第三个像素发生变化，而第二个和第四个像素不变。

显示结果的图像：http: //prntscr.com/nwl2gk

4

1 回答 1

4

该函数在一个局部变量中rand_pixel创建一个image::Image，然后返回一个指向它的指针。问题是图像在函数末尾被丢弃，所以这个指针是悬空的。

取消引用此指针是Undefined Behavior，这显然在您的应用程序中表现为某些像素被绘制了错误的颜色。删除后Image，它用来存储像素的内存可以重新用于其他任何事情。似乎大部分还没有被重新使用，但是一些数据已经写入了前几个字节。

在安全代码中，Rust 通过检查引用的生命周期是否超出被引用变量的范围来保护您免受此类错误的影响。但是，当您深入unsafe研究代码并使用原始指针时，您必须自己确保内存安全。

一种可能的解决方法是使绘图函数对 a 进行可变引用，Image而不是拥有它自己：

fn rand_pixel(pixels: &mut image::Image) -> *const std::ffi::c_void {
    // Define colors
    let black_pixel = image::Color { red: 0, green: 0, blue: 0 };
    let white_pixel = image::Color { red: 255, green: 255, blue: 255 };

    // Overwrite with a blank image
    *pixels = image::Image::new(pixels.width, pixels.height, black_pixel);

    // Get two random numbers
    use rand::Rng;
    let mut rng = rand::thread_rng();
    let rand_x = rng.gen() % pixels.width;
    let rand_y = rng.gen() % pixels.height;

    // Change pixels
    pixels.set_pixel(rand_x, rand_y, white_pixel);

    // return pointer (still valid because the image lives longer than this function)
    pixels.get_ptr()
}

在您的主要功能中，您拥有图像并通过可变引用传递它：

// this variable lives longer than the call to window::create_window
let mut pixels = image::Image::new(image_width, image_height, image::Color { red: 0, green: 0, blue: 0} );

// Define draw function
let draw_fun = || rand_pixel(&mut pixels);

// Create new window
window::create_window(1600.0, 900.0, image_width as i32, image_height as i32, draw_fun);

请注意，您还必须更改签名create_window以接受 aFnMut而不是 a Fn。

于 2019-06-02T13:13:51.320 回答