求教，ImageMagick多线程，程序崩溃

lhd666

我写了一段图片压缩的代码，这段代码在单线程的时候没发现异常，可一旦多线程运行，就会报段错误，接收到SIGSEGV信号。我使用gdb进行调试，每次where的时候，报错都不一样，可能是因为我不太会用gdb。只好贴上代码，希望大家帮我看看，是不是哪儿写的不对。这段代码运行的时候，依然会压缩出几张图片来。
操作系统：fedora25
gcc：6.2.1
ImageMagick：6.9.3

1. #include <stdio.h>
   
2. #include <stdlib.h>
   
3. #include <pthread.h>
   
4. #include <wand/magick_wand.h>
   
5. 
   
6. static int image_compress(const char *infile, const char *outfile, size_t max)
   
7. {
   
8.         MagickBooleanType status;
   
9.         MagickWand *wand;
   
10.         int width, height;
    
11.         int ret = -1;
    
12. 
    
13.         MagickWandGenesis();
    
14.         
    
15.         wand = NewMagickWand();
    
16. 
    
17.         status = MagickReadImage(wand, infile);
    
18.         if (status == MagickFalse)
    
19.                 goto out;
    
20. 
    
21.         width = MagickGetImageWidth(wand);
    
22.         height = MagickGetImageHeight(wand);
    
23. 
    
24.         if (width <= max && height <= max) {
    
25.                 ret = 1;
    
26.                 goto out;
    
27.         }
    
28. 
    
29.         if (width > height) {
    
30.                 height = height * max / width;
    
31.                 if (height == 0)
    
32.                         height = 1;
    
33.                 width = max;
    
34.         } else {
    
35.                 width = width * max / height;
    
36.                 if (width == 0)
    
37.                         width = 1;
    
38.                 height = max;
    
39.         }
    
40. 
    
41.         MagickResetIterator(wand);
    
42.         while (MagickNextImage(wand) != MagickFalse)
    
43.                 MagickResizeImage(wand, width, height, LanczosFilter, 1.0);
    
44. 
    
45.         status = MagickWriteImages(wand, outfile, MagickTrue);
    
46.         if (status == MagickFalse)
    
47.                 goto out;
    
48.         ret = 0;
    
49. 
    
50. out:
    
51.         DestroyMagickWand(wand);
    
52.         MagickWandTerminus();
    
53.         return ret;
    
54. }
    
55. 
    
56. void *thread_func(void *arg)
    
57. {
    
58.         const char *infile = arg;
    
59.         char outfile[256];
    
60. 
    
61.         for (int i = 0; i < 10; i++) {
    
62.                 snprintf(outfile, sizeof(outfile),
    
63.                          "%ld-%d", pthread_self(), i);
    
64.                 image_compress(infile, outfile, 1024);
    
65.         }
    
66. 
    
67.         return NULL;
    
68. }
    
69. 
    
70. #define ARRAY_LENGTH(a) (sizeof(a) / sizeof((a)[0]))
    
71. 
    
72. int main(int argc, char **argv)
    
73. {
    
74.         pthread_t tid[4];
    
75. 
    
76.         if (argc != 2) {
    
77.                 fprintf(stderr, "Useage: %s <input image filename>\n", argv[0]);
    
78.                 exit(EXIT_FAILURE);
    
79.         }
    
80. 
    
81.         for (int i = 0; i < ARRAY_LENGTH(tid); i++)
    
82.                 pthread_create(&tid[i], NULL, thread_func, argv[1]);
    
83. 
    
84.         for (int i = 0; i < ARRAY_LENGTH(tid); i++)
    
85.                 pthread_join(tid[i], NULL);
    
86. 
    
87.         return 0;
    
88. }

复制代码

VIP_fuck

多线程不需要用锁保护吗？

lhd666

回复 2# VIP_fuck

嗯，我就是不知道它的函数库是不是线程安全的，如果不是，由于压缩非常耗时，有的图片甚至需要十多秒（在本机找的一些较大的图片，还没特意去找大图），加锁的话很难起到提升效率的效果，达不到预期目的，如果不是线程安全的话，暂时想到的只有fork()了。

lxyscls

回复 1# lhd666

image_compress(infile, outfile, 1024)

看不出，用多线程有什么好处？把infile输出到多个outfile文件？但是也没对位置什么的做区分，不是重复工作吗？

lhd666

回复 4# lxyscls

哦，是这样的，我这个只是个示例，实际应用的时候肯定是不同的infile，将这些infile分别进行压缩，而不是对同一个infile进行多次压缩，这里只是写代码时偷了点懒。只有image_compress()才是真正需要用到的代码，其它的相当于它的测试代码。

lhd666

回复 4# lxyscls

我的测试代码里面没有使用任何全局变量，我猜测可能是ImageMagick的代码里使用了全局变量，导致多线程出问题，但我对它的库不熟，所以不太确定是不是自己代码忽略了什么步骤或者顺序不对之类的。

MMMIX

回复 6# lhd666

http://stackoverflow.com/questio ... hout-threads-option

lhd666

回复 7# MMMIX

谢谢！早知道我当初编程时直接就用多进程就好了 。

lhd666

还把内容补充到这儿，会比较方便。
ImageMagick本身是线程安全的，但是它底层调用的一些库并不是，例如当前的JPEG就不是线程安全。如果需要并行的话，建议还是用fork()。