Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Panic on empty font name #188

Closed
dm606 opened this issue Jul 20, 2018 · 1 comment
Closed

Panic on empty font name #188

dm606 opened this issue Jul 20, 2018 · 1 comment

Comments

@dm606
Copy link

dm606 commented Jul 20, 2018

Running tectonic 0.1.8 on

\documentclass{standalone}
\usepackage{fontspec}
\setmainfont{}

\begin{document}
\end{document}

crashes with

note: this is a BETA release; ask questions and report bugs at https://tectonic.newton.cx/
Running TeX ...
thread 'main' panicked at 'name must be non-empty', src/io/memory.rs:135:9
stack backtrace:
   0: std::sys::unix::backtrace::tracing::imp::unwind_backtrace
             at libstd/sys/unix/backtrace/tracing/gcc_s.rs:49
   1: std::sys_common::backtrace::print
             at libstd/sys_common/backtrace.rs:71
             at libstd/sys_common/backtrace.rs:59
   2: std::panicking::default_hook::{{closure}}
             at libstd/panicking.rs:206
   3: std::panicking::default_hook
             at libstd/panicking.rs:222
   4: std::panicking::rust_panic_with_hook
             at libstd/panicking.rs:400
   5: std::panicking::begin_panic
   6: <tectonic::io::memory::MemoryIo as tectonic::io::IoProvider>::input_open_name
   7: <tectonic::engines::ExecutionState<'a, I>>::input_open_name_format
   8: tectonic::engines::input_open
   9: tt_open_input
  10: read_font_info
  11: new_font
  12: prefixed_command
  13: main_control
  14: tt_run_engine
  15: tex_simple_main
  16: tectonic::engines::tex::TexEngine::process
  17: tectonic::driver::ProcessingSession::tex_pass
  18: tectonic::driver::ProcessingSession::default_pass
  19: tectonic::driver::ProcessingSession::run
  20: tectonic::main
  21: std::rt::lang_start::{{closure}}
  22: std::panicking::try::do_call
             at libstd/rt.rs:59
             at libstd/panicking.rs:305
  23: __rust_maybe_catch_panic
             at libpanic_unwind/lib.rs:101
  24: std::rt::lang_start_internal
             at libstd/panicking.rs:284
             at libstd/panic.rs:361
             at libstd/rt.rs:58
  25: main
  26: __libc_start_main
  27: _start
@pkgw
Copy link
Collaborator

pkgw commented Jul 23, 2018

Thanks for reporting! That's definitely a bug.

rekka added a commit to rekka/tectonic that referenced this issue Aug 27, 2018
  - Just return `OpenResult::NotAvailable`, matching the behavior of
    xelatex.

Fixes tectonic-typesetting#188.
@pkgw pkgw closed this as completed in #212 Aug 27, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants