Bug 339978 – GLib-CRITICAL **: g_dir_read_name: assertion `dir != NULL' failed

After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.

Bug 339978 - GLib-CRITICAL **: g_dir_read_name: assertion `dir != NULL' failed


Summary:	GLib-CRITICAL **: g_dir_read_name: assertion `dir != NULL' failed


Status:	RESOLVED FIXED

Product:	yelp
Classification:	Applications
Component:	Info Pages
Version:	git master
Hardware:	Other Linux

Importance:	Normal normal
Target Milestone:	---
Assigned To:	Yelp maintainers
QA Contact:	Yelp maintainers

URL:
Whiteboard:

Depends on:
Blocks:

Reported:	2006-04-28 00:42 UTC by Brent Smith (smitten)
Modified:	2006-05-11 18:25 UTC

See Also:
GNOME target:	---
GNOME version:	---

Description Brent Smith (smitten) 2006-04-28 00:42:32 UTC

Get the following after searching for "tcsh" and clicking on the "cvs info manual"

(yelp:6249): GLib-CRITICAL **: g_dir_read_name: assertion `dir != NULL' failed

(yelp:6249): GLib-CRITICAL **: g_dir_close: assertion `dir != NULL' failed

(yelp:6249): GLib-CRITICAL **: g_dir_read_name: assertion `dir != NULL' failed

(yelp:6249): GLib-CRITICAL **: g_dir_close: assertion `dir != NULL' failed



Running with G_DEBUG=fatal_criticals provides the following stacktrace:

Backtrace was generated from '/opt/gnome2/bin/yelp'

Using host libthread_db library "/lib/tls/i686/cmov/libthread_db.so.1".
[Thread debugging using libthread_db enabled]
[New Thread -1232381664 (LWP 6552)]
[New Thread -1301369936 (LWP 6565)]
[New Thread -1292977232 (LWP 6564)]
[New Thread -1284584528 (LWP 6563)]
[New Thread -1276191824 (LWP 6562)]
[New Thread -1267704912 (LWP 6554)]
[New Thread -1240110160 (LWP 6553)]
0xffffe410 in __kernel_vsyscall ()

+ Trace 67894

Thread 1 (Thread -1232381664 (LWP 6552))

#0 __kernel_vsyscall
#1 __waitpid_nocancel
from /lib/tls/i686/cmov/libpthread.so.0
#2 libgnomeui_segv_handle
at gnome-ui-init.c line 820
#3 <signal handler called>
#4 g_logv
at gmessages.c line 493
#5 g_log
at gmessages.c line 517
#6 g_return_if_fail_warning
at gmessages.c line 532
#7 g_dir_read_name
at gdir.c line 198
#8 find_info_part
at yelp-info-parser.c line 140
#9 process_indirect_map
at yelp-info-parser.c line 191
#10 yelp_info_parser_parse_file
at yelp-info-parser.c line 558
#11 info_pager_parse
at yelp-info-pager.c line 184
#12 xslt_pager_process
at yelp-xslt-pager.c line 189
#13 g_idle_dispatch
at gmain.c line 3796
#14 g_main_dispatch
at gmain.c line 1916
#15 g_main_context_dispatch
at gmain.c line 2466
#16 g_main_context_iterate
at gmain.c line 2547
#17 g_main_loop_run
at gmain.c line 2751
#18 bonobo_main
at bonobo-main.c line 311
#19 main
at yelp-main.c line 475
#0 __kernel_vsyscall

Comment 1 Don Scorgie 2006-05-11 18:25:07 UTC

*sigh*

2006-05-11  Don Scorgie  <dscorgie@cvs.gnome.org>

	* src/yelp-info-parser.c: 
	Actually check a directory exists before reading it (bug #339978)

/me smacks himself

You can also trigger this bug by loading info:cvs on its own (or any other multipart info file)

Fixed in HEAD.  Closing.