Bug 22164

Summary:	При ошибке вызывает abort() вместо exit()
Product:	Sisyphus	Reporter:	Slava Semushin <php-coder>
Component:	mpd	Assignee:	Konstantin A Lepikhov (L.A. Kostis) <lakostis>
Status:	CLOSED FIXED	QA Contact:	qa-sisyphus
Severity:	normal
Priority:	P3	CC:	erthad, lakostis
Version:	unstable
Hardware:	all
OS:	Linux

Description Slava Semushin 2009-11-04 19:14:13 MSK

mpd некорректно завершается в случае возникновения ошибок. Например, если он уже запущен и попытаться запустить его снова, то он напишет, что уже запущен и вызовет abort(), в то время как нужно вызывать exit(EXIT_FAILURE). Собственно, это не новость, что автор mpd и всей сопутствующей братии зачем-то использует abort() там где это не нужно и раньше меня это не волновало, пока я не включил создание core-файлов.

Comment 1 Alexey Rusakov 2009-11-05 16:17:55 MSK

Что-то я не наблюдаю ни одного вызова abort() в коде mpd.

Comment 2 Konstantin A Lepikhov (L.A. Kostis) 2022-07-20 12:51:13 MSK

Проверил на 0.23.8:

$ git grep -ci EXIT_FAILURE
src/Main.cxx:1
src/tag/GenParseName.cxx:1
src/win32/Win32Main.cxx:1
test/ContainerScan.cxx:4
test/DumpDatabase.cxx:2
test/DumpOgg.cxx:2
test/ParseSongFilter.cxx:1
test/ReadApeTags.cxx:3
test/RunChromaprint.cxx:3
test/RunCurl.cxx:2
test/WriteFile.cxx:3
test/dump_playlist.cxx:2
test/dump_rva2.cxx:4
test/dump_text_file.cxx:2
test/read_conf.cxx:2
test/read_mixer.cxx:3
test/read_tags.cxx:4
test/run_convert.cxx:1
test/run_decoder.cxx:3
test/run_encoder.cxx:3
test/run_filter.cxx:2
test/run_gunzip.cxx:2
test/run_gzip.cxx:2
test/run_inotify.cxx:2
test/run_input.cxx:2
test/run_neighbor_explorer.cxx:2
test/run_normalize.cxx:1
test/run_output.cxx:1
test/run_resolver.cxx:2
test/run_storage.cxx:5
test/software_volume.cxx:2
test/test_vorbis_encoder.cxx:1
test/visit_archive.cxx:3