[devel] BUG: uw-imap locking does not work on reiserfs

[devel] BUG: uw-imap locking does not work on reiserfs

[Sergey Vlasov]

Внимание, грабли!

В uw-imap (по крайней мере, в версии uw-imap-2001a-alt6) не работает
блокировка почтовых ящиков, если они находятся на файловой системе
reiserfs (проверено с ядром 2.4.18-up-alt8). На ext2, ext3 блокировка
работает. На других файловых системах тоже могут быть проблемы - не
проверял.

Ошибка может приводить к порче почтовых ящиков и потере сообщений.

Причина, похоже, в следующем куске кода (src/osdep/unix/nfstnew.c):

/* Test for NFS
 * Accepts: file descriptor
 * Returns: T if NFS file, NIL otherwise
 */

long test_nfs (int fd)
{
  struct stat sbuf;
  struct ustat usbuf;
  struct statvfs vsbuf;
  /* Any base type that begins with "nfs" or "afs" is considered to be a
   * network filesystem.
   */
  return ((!fstat (fd,&sbuf) && !ustat (sbuf.st_dev,&usbuf) &&
	   !++usbuf.f_tinode) ||
	  (!fstatvfs (fd,&vsbuf) && (vsbuf.f_basetype[1] == 'f') &&
	   (vsbuf.f_basetype[2] == 's') &&
	   ((vsbuf.f_basetype[0] == 'n') || (vsbuf.f_basetype[0] == 'a')))) ?
	     LONGT : NIL;
}

На reiserfs функция ustat возвращает f_tinode == -1, в результате чего
uw-imap считает, что работает с NFS, и прекращает использовать
блокировки через fcntl (остается только dotlock, но этот метод не
работает с системными ящиками в /var/spool/mail).

-- 
Sergey Vlasov

Re: [devel] BUG: uw-imap locking does not work on reiserfs

[Ivan Zakharyaschev]

	Hello!

On Sat, 12 Oct 2002, Sergey Vlasov wrote:

Спасибо за сообщение.

> В uw-imap (по крайней мере, в версии uw-imap-2001a-alt6) не работает
> блокировка почтовых ящиков, если они находятся на файловой системе
> reiserfs (проверено с ядром 2.4.18-up-alt8). На ext2, ext3 блокировка
> работает. На других файловых системах тоже могут быть проблемы - не
> проверял.

Проверено c 2.4.19-alt0.2-up: на xfs, jfs, как уже упомяналось, ext2,
ext3, и даже vfat проблем с блокировками не замечено. На reiserfs-3.6 и
3.5 блокировка не работает. (Использование jfs оставило неблагоприятное
впечатление.)

> Ошибка может приводить к порче почтовых ящиков и потере сообщений.
>
> Причина, похоже, в следующем куске кода (src/osdep/unix/nfstnew.c):


Скоро будут выпущены пакеты с исправлениями. Ошибка проявляется при
использовании imapd, ipop{2,3}d, pine.


Ещё немного сравнительной информации, чтобы не вводить никого в
заблуждение по поводу актуальности этой ошибки для других дистрибутивов:


В RH и Mdk свой эмулятор flock(2) на основе fcntl(2), который в отличие
от ALT и оригинального (того, что идёт в составе UW imap) не занимается
этой непродуманной проверкой на NFS.

В Debian (и, судя по всему, SuSE) используется flock(2) (+
привилегированный dot-locker) -- там такой проверки просто нет, потому
что известно, что flock(2) и так не работает по NFS. (Причём такое
поведение, скорее всего, идёт вразрез с определённой Debian'овской
policy.)

(Здесь говорится о трёх возможных способах установки блокировок на
файлы-почтовые ящики: flock(2), fcntl(2)/lockf(3) и dot-locking.)

-- 
С наилучшими пожеланиями,
Иван Захарьящев, Москва

fixed: Re: [devel] BUG: uw-imap locking does not work on reiserfs

[Ivan Zakharyaschev]

	Hello!

Пакеты uw-imap и pine с исправлениями

для Сизифа лежат в стандартном месте
altair:/user/INCOMING/OUT/Sisyphus/{S,}RPMS/

для updates в altair:/user/imz/incoming/updates/SRPMS/
они требует адаптации и пересборки в средах Spring2001, Castle,
Master2.0.

Прежде чем выкладывать в updates хорошо бы их потестировать. Я сообщу о
результатах моих тестов.

-- 
С наилучшими пожеланиями,
Иван Захарьящев, Москва

Re: fixed: Re: [devel] BUG: uw-imap locking does not work on reiserfs

[Ivan Zakharyaschev]

	Hello!

On Sat, 19 Oct 2002, Ivan Zakharyaschev wrote:

> Пакеты uw-imap и pine с исправлениями

Я потестировал блокировки на всех обычных файловых системах --
теперь работают. На NFS не тестировал.

> для Сизифа лежат в стандартном месте
> altair:/user/INCOMING/OUT/Sisyphus/{S,}RPMS/
>
> для updates в altair:/user/imz/incoming/updates/SRPMS/
> они требует адаптации и пересборки в средах Spring2001, Castle,
> Master2.0.
>
> Прежде чем выкладывать в updates хорошо бы их потестировать. Я сообщу
> о
> результатах моих тестов.

-- 
С наилучшими пожеланиями,
Иван Захарьящев, Москва