LINUX.ORG.RU

История изменений

Исправление AITap, (текущая версия) :

буду рад услышать замечания

pids=`ps x | grep libflashplayer.so | grep -v grep | awk '{print $1}'`

Во-первых, grep | awk легко заменяется либо на grep | cut, либо просто на awk. Потратьте немного времени на чтение документации по этому языку (предку Perl, между прочим).
Во-вторых, то же самое делает команда pgrep -f libflashplayer.so, и уродливая фильтрация grep не нужна.

fds=`lsof -p $pid | grep Flash | \
awk '{print $4"|«$7}' | sed 's/[^0-9]//'`
...

Не проще ли `ls -l /proc/$pid/fd | awk '/Flash/{print $9}'`? Сразу получаете список файлов.

`$act`

Опасайтесь спецсимволов в именах файлов! Впрочем, если бы они были, они бы всё испортили гораздо раньше, ещё на этапе разрезания списка при помощи for var in $string.

Исходная версия AITap, :

буду рад услышать замечания
pids=`ps x | grep libflashplayer.so | grep -v grep | awk '{print $1}'`

Во-первых, grep | awk легко заменяется либо на grep | cut, либо просто на awk. Потратьте немного времени на чтение документации по этому языку (предку Perl, между прочим).
Во-вторых, то же самое делает команда pgrep -f libflashplayer.so, и уродливая фильтрация grep не нужна.

fds=`lsof -p $pid | grep Flash | \
awk '{print $4"|«$7}' | sed 's/[^0-9]//'`
...

Не проще ли `ls -l /proc/$pid/fd | awk '/Flash/{print $9}'`? Сразу получаете список файлов.

`$act`

Опасайтесь спецсимволов в именах файлов! Впрочем, если бы они были, они бы всё испортили гораздо раньше, ещё на этапе разрезания списка при помощи for var in $string.