Fossil SCM
Fix popen2() return value check in hook_backoffice().
Commit
e6aaeb1b6df6424164ddbc3a11703275ae9c60c3c17af6abb47e4d38346187a5
Parent
d3acca57b974e7d…
1 file changed
+1
-1
+1
-1
| --- src/hook.c | ||
| +++ src/hook.c | ||
| @@ -476,11 +476,11 @@ | ||
| 476 | 476 | int childPid; |
| 477 | 477 | if( cnt==0 ){ |
| 478 | 478 | hook_changes(&chng, zLastRcvid, 0); |
| 479 | 479 | } |
| 480 | 480 | zCmd = hook_subst(db_column_text(&q,0), 0); |
| 481 | - if( popen2(zCmd, &fdFromChild, &toChild, &childPid, 0) ){ | |
| 481 | + if( popen2(zCmd, &fdFromChild, &toChild, &childPid, 0)==0 ){ | |
| 482 | 482 | if( toChild ){ |
| 483 | 483 | fwrite(blob_buffer(&chng),1,blob_size(&chng),toChild); |
| 484 | 484 | } |
| 485 | 485 | pclose2(fdFromChild, toChild, childPid); |
| 486 | 486 | } |
| 487 | 487 |
| --- src/hook.c | |
| +++ src/hook.c | |
| @@ -476,11 +476,11 @@ | |
| 476 | int childPid; |
| 477 | if( cnt==0 ){ |
| 478 | hook_changes(&chng, zLastRcvid, 0); |
| 479 | } |
| 480 | zCmd = hook_subst(db_column_text(&q,0), 0); |
| 481 | if( popen2(zCmd, &fdFromChild, &toChild, &childPid, 0) ){ |
| 482 | if( toChild ){ |
| 483 | fwrite(blob_buffer(&chng),1,blob_size(&chng),toChild); |
| 484 | } |
| 485 | pclose2(fdFromChild, toChild, childPid); |
| 486 | } |
| 487 |
| --- src/hook.c | |
| +++ src/hook.c | |
| @@ -476,11 +476,11 @@ | |
| 476 | int childPid; |
| 477 | if( cnt==0 ){ |
| 478 | hook_changes(&chng, zLastRcvid, 0); |
| 479 | } |
| 480 | zCmd = hook_subst(db_column_text(&q,0), 0); |
| 481 | if( popen2(zCmd, &fdFromChild, &toChild, &childPid, 0)==0 ){ |
| 482 | if( toChild ){ |
| 483 | fwrite(blob_buffer(&chng),1,blob_size(&chng),toChild); |
| 484 | } |
| 485 | pclose2(fdFromChild, toChild, childPid); |
| 486 | } |
| 487 |