Answering your questions
Is it correct to invoke a model from another model?
If by right you mean if technically CodeIgniter allows it: Yes, from CodeIgniter 2.0 it is allowed to load models into another model.
Whether it should be done or not depends on you, however CodeIgniter is an MVC framework, so it is recommended to load the models inside the controllers and then do all the business logic, therefore if you want to be strict in the use of MVC you should load them into the controller.
Does my security affect anything?
It would not have to affect unless in the model you are using bad practices in terms of SQL security, in this entry you can read something about SQL injection How to avoid SQL injection in PHP?
Is it anti-functional?
You say it yourself
It worked perfectly for me.
However from the point of view of MVC you should not do so.
As a personal criterion I would tell you to follow what the MVC pattern says, otherwise it would not make much sense to use the framework.