Bug 221266 - www/chromium: segfaults immediately on FreeBSD 11.1/i386
Summary: www/chromium: segfaults immediately on FreeBSD 11.1/i386
Status: Closed FIXED
Alias: None
Product: Ports & Packages
Classification: Unclassified
Component: Individual Port(s) (show other bugs)
Version: Latest
Hardware: i386 Any
: --- Affects Only Me
Assignee: Carlos J. Puga Medina
URL: https://forums.freebsd.org/threads/62...
Keywords:
: 222119 (view as bug list)
Depends on:
Blocks:
 
Reported: 2017-08-06 05:48 UTC by Patrick McMunn
Modified: 2017-12-27 23:50 UTC (History)
3 users (show)

See Also:
cpm: maintainer-feedback+

Attachments
patch-chromium-libc++.diff (14.77 KB, patch)
2017-09-11 02:15 UTC, Carlos J. Puga Medina 		no flags Details | Diff
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Patrick McMunn 2017-08-06 05:48:48 UTC 
The 32-bit version of Chromium segfaults immediately on FreeBSD 11.1. It worked on FreeBSD 11.0. The 64-bit version appears to be unaffected. The only output from dmesg is "pid 51737 (chrome), uid 1001: exited on signal 11". I've only tested chromium-59.0.3071.115_1 and chromium-59.0.3071.115_2, so I don't know if other versions have been affected.
Comment 1 Carlos J. Puga Medina freebsd_committer freebsd_triage 2017-09-04 17:05:17 UTC 
(In reply to Patrick from comment #0)

Are you able to reproduce the segfault after update chromium to 60?
Comment 2 Carlos J. Puga Medina freebsd_committer freebsd_triage 2017-09-07 11:23:06 UTC 
*** Bug 222119 has been marked as a duplicate of this bug. ***
Comment 3 Carlos J. Puga Medina freebsd_committer freebsd_triage 2017-09-07 11:40:59 UTC 
I confirm that only chromium 32-bit version is affected.

I will try to amend this issue in the next chromium update to 61.
Comment 4 Carlos J. Puga Medina freebsd_committer freebsd_triage 2017-09-11 02:15:11 UTC 
Created attachment 186237 [details]
patch-chromium-libc++.diff

The attached patch should fix the segfaults on i386.

Please, report back after checking the result.
Comment 5 commit-hook freebsd_committer freebsd_triage 2017-09-17 11:11:00 UTC 
A commit references this bug:

Author: cpm
Date: Sun Sep 17 11:10:46 UTC 2017
New revision: 449991
URL: https://svnweb.freebsd.org/changeset/ports/449991

Log:
  www/chromium: update to 61.0.3163.79

  - Add arm64 support [1]
  - Fix segfault on i386 [2]
  - Switch to USES=compiler:c++14-lang for devel/chromium-gn because chromium
    needs __builtin_add_overflow and friends to build.

  Changelog: https://chromium.googlesource.com/chromium/src/+log/60.0.3112.113..61.0.3163.79?pretty=fuller&n=10000

  PR:		220291 [1], 221266 [2]
  Submitted by:	Andrew [1]
  Reported by:	Patrick <doctorwhoguy@gmail.com> [2]
  MFH:		2017Q3

Changes:
  head/www/chromium/Makefile
  head/www/chromium/distinfo
  head/www/chromium/files/extra-patch-libc++-new
  head/www/chromium/files/extra-patch-libc++-old
  head/www/chromium/files/patch-base_i18n_icu__util.cc
  head/www/chromium/files/patch-base_process_memory__unittest.cc
  head/www/chromium/files/patch-base_process_process__metrics__posix.cc
  head/www/chromium/files/patch-base_sys__info__posix.cc
  head/www/chromium/files/patch-base_task_scheduler_delayed_task_manager.cc
  head/www/chromium/files/patch-base_test_test__file__util__posix.cc
  head/www/chromium/files/patch-breakpad_BUILD.gn
  head/www/chromium/files/patch-build_config_BUILDCONFIG.gn
  head/www/chromium/files/patch-build_config_compiler_BUILD.gn
  head/www/chromium/files/patch-build_toolchain_linux_BUILD.gn
  head/www/chromium/files/patch-chrome_BUILD.gn
  head/www/chromium/files/patch-chrome_app_chrome__main.cc
  head/www/chromium/files/patch-chrome_app_chrome__main__delegate.cc
  head/www/chromium/files/patch-chrome_app_chromium__strings.grd
  head/www/chromium/files/patch-chrome_app_generated__resources.grd
  head/www/chromium/files/patch-chrome_app_google__chrome__strings.grd
  head/www/chromium/files/patch-chrome_app_theme_chrome_unscaled_resources.grd
  head/www/chromium/files/patch-chrome_browser_background_background_mode_optimizer.cc
  head/www/chromium/files/patch-chrome_browser_browser__process__impl.cc
  head/www/chromium/files/patch-chrome_browser_browser__resources.grd
  head/www/chromium/files/patch-chrome_browser_chrome__content__browser__client.cc
  head/www/chromium/files/patch-chrome_browser_download_download__commands.h
  head/www/chromium/files/patch-chrome_browser_extensions_BUILD.gn
  head/www/chromium/files/patch-chrome_browser_extensions_browser__context__keyed__service__factories.cc
  head/www/chromium/files/patch-chrome_browser_flag__descriptions.cc
  head/www/chromium/files/patch-chrome_browser_media_router_discovery_discovery_network_list_posix.cc
  head/www/chromium/files/patch-chrome_browser_media_router_discovery_discovery_network_list_wifi_linux.cc
  head/www/chromium/files/patch-chrome_browser_metrics_chrome__browser__main__extra__parts__metrics.cc
  head/www/chromium/files/patch-chrome_browser_password__manager_chrome__password__manager__client.cc
  head/www/chromium/files/patch-chrome_browser_prefs_browser_prefs.cc
  head/www/chromium/files/patch-chrome_browser_profiles_chrome_browser_main_extra_parts_profiles.cc
  head/www/chromium/files/patch-chrome_browser_ui_browser__view__prefs.cc
  head/www/chromium/files/patch-chrome_browser_ui_views_apps_chrome__native__app__window__views__aura.cc
  head/www/chromium/files/patch-chrome_browser_ui_views_profiles_avatar_button.cc
  head/www/chromium/files/patch-chrome_browser_ui_views_tabs_new_tab_button.cc
  head/www/chromium/files/patch-chrome_browser_ui_views_tabs_tab__strip.cc
  head/www/chromium/files/patch-chrome_browser_ui_webui_about__ui.cc
  head/www/chromium/files/patch-chrome_common_chrome__features.cc
  head/www/chromium/files/patch-chrome_common_chrome__features.h
  head/www/chromium/files/patch-chrome_common_features.gni
  head/www/chromium/files/patch-chrome_common_url__constants.cc
  head/www/chromium/files/patch-chrome_common_url__constants.h
  head/www/chromium/files/patch-components_autofill_content_renderer_password__form__conversion__utils.cc
  head/www/chromium/files/patch-components_content__settings_core_browser_website__settings__registry.cc
  head/www/chromium/files/patch-components_content__settings_core_browser_website__settings__registry.h
  head/www/chromium/files/patch-components_os__crypt_os__crypt.h
  head/www/chromium/files/patch-components_password__manager_core_browser_BUILD.gn
  head/www/chromium/files/patch-components_password__manager_core_browser_password_manager_metrics_util.cc
  head/www/chromium/files/patch-components_password__manager_core_browser_password_manager_metrics_util.h
  head/www/chromium/files/patch-components_password__manager_core_browser_password_reuse_defines.h
  head/www/chromium/files/patch-components_policy_BUILD.gn
  head/www/chromium/files/patch-components_policy_resources_policy__templates.json
  head/www/chromium/files/patch-components_policy_tools_generate__policy__source.py
  head/www/chromium/files/patch-components_storage__monitor_storage__monitor.cc
  head/www/chromium/files/patch-components_tracing_common_process__metrics__memory__dump__provider.cc
  head/www/chromium/files/patch-components_tracing_common_process__metrics__memory__dump__provider.h
  head/www/chromium/files/patch-components_variations_proto_study.proto
  head/www/chromium/files/patch-content_browser_BUILD.gn
  head/www/chromium/files/patch-content_browser_accessibility_browser__accessibility.h
  head/www/chromium/files/patch-content_browser_accessibility_browser__accessibility__manager.h
  head/www/chromium/files/patch-content_browser_accessibility_browser_accessibility_auralinux.cc
  head/www/chromium/files/patch-content_browser_zygote_host_zygote_host_impl_linux.h
  head/www/chromium/files/patch-content_public_browser_zygote_host_linux.h
  head/www/chromium/files/patch-content_renderer_render__thread__impl.cc
  head/www/chromium/files/patch-content_renderer_renderer_main.cc
  head/www/chromium/files/patch-content_shell_BUILD.gn
  head/www/chromium/files/patch-crypto_nss__util.cc
  head/www/chromium/files/patch-device_gamepad_BUILD.gn
  head/www/chromium/files/patch-headless_lib_browser_headless__content__browser__client.h
  head/www/chromium/files/patch-ipc_ipc__message__utils.cc
  head/www/chromium/files/patch-ipc_ipc__message__utils.h
  head/www/chromium/files/patch-net_dns_host__resolver__proc.cc
  head/www/chromium/files/patch-net_features.gni
  head/www/chromium/files/patch-printing_backend_print_backend_cups.h
  head/www/chromium/files/patch-remoting_host_setup_me2me_native_messaging_host_main.cc
  head/www/chromium/files/patch-sdch_BUILD.gn
  head/www/chromium/files/patch-services_catalog_public_cpp_manifest__parsing__util.cc
  head/www/chromium/files/patch-services_catalog_store.cc
  head/www/chromium/files/patch-services_catalog_store.h
  head/www/chromium/files/patch-services_device_time__zone__monitor_time__zone__monitor.cc
  head/www/chromium/files/patch-services_resource_coordinator_public_cpp_memory_instrumentation_os_metrics.h
  head/www/chromium/files/patch-services_resource_coordinator_public_cpp_memory_instrumentation_os_metrics_linux.cc
  head/www/chromium/files/patch-third__party_WebKit_Source_bindings_core_v8_V8ScriptRunner.cpp
  head/www/chromium/files/patch-third__party_WebKit_Source_core_editing_EditingBehavior.cpp
  head/www/chromium/files/patch-third__party_WebKit_Source_core_html_forms_InternalPopupMenu.cpp
  head/www/chromium/files/patch-third__party_WebKit_Source_platform_fonts_FontCache.cpp
  head/www/chromium/files/patch-third__party_WebKit_Source_platform_fonts_FontCache.h
  head/www/chromium/files/patch-third__party_WebKit_Source_platform_fonts_FontPlatformData.cpp
  head/www/chromium/files/patch-third__party_WebKit_Source_platform_fonts_FontPlatformData.h
  head/www/chromium/files/patch-third__party_WebKit_Source_platform_fonts_SimpleFontData.cpp
  head/www/chromium/files/patch-third__party_WebKit_Source_platform_fonts_skia_FontCacheSkia.cpp
  head/www/chromium/files/patch-third__party_WebKit_Source_platform_heap_StackFrameDepth.cpp
  head/www/chromium/files/patch-third__party_WebKit_Source_platform_wtf_Assertions.cpp
  head/www/chromium/files/patch-third__party_WebKit_Source_platform_wtf_Atomics.h
  head/www/chromium/files/patch-third__party_WebKit_Source_platform_wtf_ContainerAnnotations.h
  head/www/chromium/files/patch-third__party_WebKit_Source_platform_wtf_StackUtil.cpp
  head/www/chromium/files/patch-third__party_WebKit_Source_platform_wtf_ThreadingPthreads.cpp
  head/www/chromium/files/patch-third__party_WebKit_Source_web_PopupMenuImpl.cpp
  head/www/chromium/files/patch-third__party_angle_BUILD.gn
  head/www/chromium/files/patch-third__party_angle_src_gpu_info_util_SystemInfo_internal.h
  head/www/chromium/files/patch-third__party_angle_src_gpu_info_util_SystemInfo_linux.cpp
  head/www/chromium/files/patch-third__party_angle_src_gpu_info_util_SystemInfo_x11.cpp
  head/www/chromium/files/patch-third__party_boringssl_BUILD.gn
  head/www/chromium/files/patch-third__party_boringssl_src_crypto_cpu-aarch64-linux.c
  head/www/chromium/files/patch-third__party_ffmpeg_chromium_scripts_generate_gn.py
  head/www/chromium/files/patch-third__party_ffmpeg_ffmpeg__generated.gni
  head/www/chromium/files/patch-third__party_fontconfig_BUILD.gn
  head/www/chromium/files/patch-third__party_fontconfig_fontconfig.gni
  head/www/chromium/files/patch-third__party_libxml_chromium_libxml__utils.cc
  head/www/chromium/files/patch-third__party_pdfium_core_fxge_fx_ge_linux.cpp
  head/www/chromium/files/patch-third__party_pdfium_fpdfsdk_javascript_JS__Value.cpp
  head/www/chromium/files/patch-third__party_pdfium_xfa_fgas_font_cfgas_fontmgr.cpp
  head/www/chromium/files/patch-third__party_swiftshader_src_OpenGL_libEGL_EGLSurface.cpp
  head/www/chromium/files/patch-third__party_swiftshader_src_OpenGL_libEGL_Surface.cpp
  head/www/chromium/files/patch-third__party_swiftshader_src_OpenGL_libEGL_libEGL.cpp
  head/www/chromium/files/patch-third__party_webrtc_base_BUILD.gn
  head/www/chromium/files/patch-third__party_webrtc_base_byteorder.h
  head/www/chromium/files/patch-third__party_webrtc_base_httpcommon.cc
  head/www/chromium/files/patch-third__party_webrtc_base_ifaddrs__converter.h
  head/www/chromium/files/patch-third__party_webrtc_base_ipaddress.h
  head/www/chromium/files/patch-third__party_webrtc_base_network.cc
  head/www/chromium/files/patch-third__party_webrtc_base_network.h
  head/www/chromium/files/patch-third__party_webrtc_base_physicalsocketserver.cc
  head/www/chromium/files/patch-third__party_webrtc_base_platform__thread.cc
  head/www/chromium/files/patch-third__party_webrtc_base_stringutils.h
  head/www/chromium/files/patch-third__party_webrtc_rtc_base_BUILD.gn
  head/www/chromium/files/patch-third__party_webrtc_rtc_base_byteorder.h
  head/www/chromium/files/patch-third__party_webrtc_rtc_base_httpcommon.cc
  head/www/chromium/files/patch-third__party_webrtc_rtc_base_ifaddrs__converter.h
  head/www/chromium/files/patch-third__party_webrtc_rtc_base_ipaddress.h
  head/www/chromium/files/patch-third__party_webrtc_rtc_base_network.cc
  head/www/chromium/files/patch-third__party_webrtc_rtc_base_network.h
  head/www/chromium/files/patch-third__party_webrtc_rtc_base_physicalsocketserver.cc
  head/www/chromium/files/patch-third__party_webrtc_rtc_base_physicalsocketserver.h
  head/www/chromium/files/patch-third__party_webrtc_rtc_base_platform_thread.cc
  head/www/chromium/files/patch-third__party_webrtc_rtc_base_stringutils.h
  head/www/chromium/files/patch-tools_gn_bootstrap_bootstrap.py
  head/www/chromium/files/patch-tools_grit_grit__rule.gni
  head/www/chromium/files/patch-ui_base_BUILD.gn
  head/www/chromium/files/patch-ui_base_resource_resource__bundle__freebsd.cc
  head/www/chromium/files/patch-ui_gfx_BUILD.gn
  head/www/chromium/files/patch-ui_gfx_mojo_buffer_types_struct_traits.cc
  head/www/chromium/files/patch-ui_gl_BUILD.gn
  head/www/chromium/files/patch-ui_gl_generate_bindings.py
  head/www/chromium/files/patch-ui_native_theme_native_theme.h
  head/www/chromium/files/patch-ui_native_theme_native_theme_base.cc
  head/www/chromium/files/patch-ui_resources_ui__resources.grd
  head/www/chromium/files/patch-ui_strings_app_locale_settings.grd
  head/www/chromium/files/patch-ui_views_BUILD.gn
  head/www/chromium/files/patch-ui_views_mus_aura__init.cc
  head/www/chromium/files/patch-ui_views_views__switches.cc
  head/www/chromium/files/patch-ui_views_window_frame_background.cc
  head/www/chromium/files/patch-v8_src_base_cpu.cc
  head/www/chromium/pkg-plist
Comment 6 Carlos J. Puga Medina freebsd_committer freebsd_triage 2017-09-23 02:16:49 UTC 
Reopened because the problem has not been solved yet.
Comment 7 megiddgit 2017-09-29 08:44:58 UTC 
Just wanted to confirm, on my 32-bit machine, chromium 58 works fine but chromium 59, 60, 61 give segmentation fault.

$ uname -a
FreeBSD D820 10.3-STABLE FreeBSD 10.3-STABLE #0 r300092: Wed May 18 02:02:40 UTC 2016     root@releng1.nyi.freebsd.org:/usr/obj/usr/src/sys/GENERIC  i386

Thanks for maintaining Chromium.
Comment 8 Carlos J. Puga Medina freebsd_committer freebsd_triage 2017-12-27 23:49:51 UTC 
Fixed in r457431.