yaml-cpp crashed when loading a wrong format yaml file

[HensonLi]

When using YAML::Node rNode = YAML::LoadFile(“test.yaml”)

• terminal output log as below :
  terminating with uncaught exception of type YAML::ParserException: yaml-cpp: error at line 14, column 15: illegal map value

• test.yaml's content as below:

com.hh.example:
permission: 1,2
bookManager:
descriptor: com.hh.example.service.BookManager
addMsg:
methodNumber: 2
permission: 1,2
getFrontMsg:
methodNumber: 1
permission: 1
com.hh.msp.client:
permission: 1,2
test:henson`

( sorry for that i can't make the file content indent correctly)

After checking the code, i found that the method Scanner::ScanValue() throw out a execption with throw ParserException(INPUT.mark(), ErrorMsg::MAP_VALUE);, but seems no code the handle this execption

Question :

1. seems than the YAML::Parser can not parse or skip test:henson which has no black space after ":"
2. Shoud there be one execption to handle such issue other that let it crash

[tt4g]

I'm not good at English, so I may have misunderstood your comment.

seems than the YAML::Parser can not parse or skip test:henson which has no black space after ":"

test: henson can not be parsed as YAML, You have to add whitespace (i.e space) after ":".

Shoud there be one execption to handle such issue other that let it crash

This error is not handled inside yaml-cpp because it is a syntax error in the your YAML file.
You need to handle exception.

example:

try {
    YAML::Node rNode = YAML::LoadFile(“test.yaml”);

    // using YAML node.

} catch(const YAML::ParserException& ex) {
    std::cout << ex.what() << std::endl;
}

see:

yaml-cpp/include/yaml-cpp/node/parse.h

Lines 40 to 46 in 4fb1c4b

	/**
	* Loads the input file as a single YAML document.
	*
	* @throws {@link ParserException} if it is malformed.
	* @throws {@link BadFile} if the file cannot be loaded.
	*/
	YAML_CPP_API Node LoadFile(const std::string& filename);

[HensonLi]

@tt4g Thanks for your reply, you have totally understood what i am saying, there is no problem about catch the execption myself, just recommend to handle it in yaml-cpp to prevent to programe from crash.

[tt4g]

@HensonLi OK. The addition of an API that does not generate exception is closely related to the design of yaml-cpp.

Let's wait for @jbeder to confirm this issue.

[jbeder]

@tt4g is correct; this is working as intended. The library is designed for you to handle incorrectly formed YAML by catching ParserException.