Merge pull request diffblue#2421 from tautschnig/vs-bound

tautschnig · web-flow · commit f27e7242b400 · 2018-06-23T21:27:39.000+01:00
Bound the number of attempts my_mkstemps tries to compute a file name
diff --git a/src/util/tempfile.cpp b/src/util/tempfile.cpp
@@ -62,7 +62,7 @@ int my_mkstemps(char *template_str, int suffix_len)
   static long long unsigned int random_state;
   random_state+=getpid()+123;
 
-  for(unsigned attempt=0; ; ++attempt)
+  for(unsigned attempt = 0; attempt < 1000; ++attempt)
   {
     unsigned long long number=random_state;
 

Original file line number	Diff line number	Diff line change
`@@ -62,7 +62,7 @@ int my_mkstemps(char *template_str, int suffix_len)`
`62`	`62`	`static long long unsigned int random_state;`
`63`	`63`	`random_state+=getpid()+123;`
`64`	`64`
`65`		`- for(unsigned attempt=0; ; ++attempt)`
	`65`	`+ for(unsigned attempt = 0; attempt < 1000; ++attempt)`
`66`	`66`	`{`
`67`	`67`	`unsigned long long number=random_state;`
`68`	`68`