PHP

質問

<?php
$tmpfname = tempnam("/var/tmp", "auto_pbxrestart");

$cmd = "/usr/bin/sudo chown 777 ".TEMP_PATH;
exec($cmd, $output, $return);
$cmd = "/usr/bin/sudo chown apache::apache ".TEMP_PATH:
exec($cmd, $output, $return);

$handle = fopen($tmpfname, "w")

if ($handle == false) {

$status = '01';
print "debug02err";

}

/* 実際にクーロンに書き込みする（「* *」は月日なので今回は設定対象にならない）*
$cron_config = $arr['opt'] . $arr['nminute'] . " " . $arr['nhour'] . " * * " .$arr['days'] . " root /usr/local/sbin/auto_pbxrestart.pl";
fwrite($fpt , $cron_config);

fclose($fpt);

/* テンポラリのクーロンの設定をして本番のクーロンに設定をする */
print "debug04=".$cmd.":".$return."<br>";
$cmd = "/usr/bin/sudo cp -f " . CRON_DAILY_ESTAB . " " . CRON_DAILY_ESTAB . ".bak";
exec($cmd, $output, $return);
$cmd = "/usr/bin/sudo cp -f " . TEMP_PATH . " " . CRON_DAILY_ESTAB;
exec($cmd, $output, $return);
unlink($tmpfname);
print "debug05 = ".$cmd.":".$return."<br>";
/* 正常終了 */

?>

回答No.1

t_ohta · ベストアンサー

質問内容が書かれていませんが、何が問題なのでしょう。

お礼

ありがとうございます^_^

回答No.1

AsarKingChang

$tmpfname = tempnam("/var/tmp", "auto_pbxrestart");
~~
~~ $cmd = "/usr/bin/sudo chown 777 ".TEMP_PATH;
~~ exec($cmd, $output, $return);
~~ $cmd = "/usr/bin/sudo chown apache::apache ".TEMP_PATH:
~~ exec($cmd, $output, $return);

$handle = fopen($tmpfname, "w")

質問は書かれてないのだが、
～～のくだりいらないですよ。
最初から、テンポラリファイルを作る命令があります。
https://www.php.net/manual/ja/function.tmpfile.php
基本、execに頼るのをやめれば、解決する問題がたくさんありそうですが。
ただし、
$cmd = "/usr/bin/sudo chown apache::apache ".TEMP_PATH:
これも、同じでApache権限かApacheからアクセスできる権限で作ったファイルなら、
オーナーをapacheにする必要はないですよ。
すでに「書けた」のであれば、元々apacheからアクセス権限がある状態なので。
これは、不要です。

$cmd = "/usr/bin/sudo chown 777 ".TEMP_PATH;
これも、間違い、chownはオーナー変更ですが、パラメーターがパーミッション値です。
https://www.php.net/manual/ja/function.chmod.php
やりたいならそもそも専用の関数があるので、そちらを使うほうが
try~catchでエラーが出た時理由などもデバッガーに出るので、
より早い解決ができるかと。

という事で、使っている命令がそもそも、
違うのを治してみませんか？

あと、テンポラリでクーロンを作って、ユーザー権限で動くクーロンにポーリングさせれば、
移動だの更新だのっていらないですよ。
https://qiita.com/iganari/items/5a61ec93d989f7c77a2c
こっちを使えばほとんどの問題は解決すると思います。

（そもそも、PHPから/etcなどにアクセスしてる限り、
　ストレスたまるだけですよ）